This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] x86: Update VFIXUPIMM* Intrinsics to align with the latest Intel SDM


On Sun, Nov 4, 2018 at 11:00 PM Uros Bizjak <ubizjak@gmail.com> wrote:
>
> On Mon, Nov 5, 2018 at 6:54 AM Wei Xiao <wei.william.xiao@gmail.com> wrote:
> >
> > > Please also rename these:
> > >
> > >  _mm512_mask_fixupimm_round_pd (__m512d __A, __mmask8 __U, __m512d __B,
> > >                     __m512i __C, const int __imm, const int __R)
> > >
> > >  _mm512_mask_fixupimm_round_ps (__m512 __A, __mmask16 __U, __m512 __B,
> > >                     __m512i __C, const int __imm, const int __R)
> > >
> > >  _mm_mask_fixupimm_round_sd (__m128d __A, __mmask8 __U, __m128d __B,
> > >                  __m128i __C, const int __imm, const int __R)
> > >
> > >  _mm_mask_fixupimm_round_ss (__m128 __A, __mmask8 __U, __m128 __B,
> > >                  __m128i __C, const int __imm, const int __R)
> > >
> > >  _mm512_mask_fixupimm_pd (__m512d __A, __mmask8 __U, __m512d __B,
> > >               __m512i __C, const int __imm)
> > >
> > > _mm512_mask_fixupimm_ps (__m512 __A, __mmask16 __U, __m512 __B,
> > >               __m512i __C, const int __imm)
> > >
> > >  _mm_mask_fixupimm_sd (__m128d __A, __mmask8 __U, __m128d __B,
> > >                __m128i __C, const int __imm)
> > >
> > >  _mm_mask_fixupimm_ss (__m128 __A, __mmask8 __U, __m128 __B,
> > >                __m128i __C, const int __imm)
> > >
> > >  _mm256_mask_fixupimm_pd (__m256d __A, __mmask8 __U, __m256d __B,
> > >               __m256i __C, const int __imm)
> > >
> > >  _mm256_mask_fixupimm_ps (__m256 __A, __mmask8 __U, __m256 __B,
> > >               __m256i __C, const int __imm)
> > >
> > >   _mm_mask_fixupimm_pd (__m128d __A, __mmask8 __U, __m128d __B,
> > >                __m128i __C, const int __imm)
> > >
> > >  _mm_mask_fixupimm_ps (__m128 __A, __mmask8 __U, __m128 __B,
> > >                __m128i __C, const int __imm)
> > >
> > > Uros.
> >
> > As attached, I have renamed above intrinsics according to
> > aforementioned convention:
> >
> > [ __m512. __W,] __mmask. __U, __m512x __A, __m512x __B, ..., const int
> > _imm, const int __R].
>
> LGTM.
>

LGTM.

Thanks.


-- 
H.J.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]