[PATCH] D15851: [AVX512] add PSRAD and PSRAQ Intrinsic

[Asaf Badouh via llvm-commits]

AsafBadouh added inline comments.

================
Comment at: llvm/trunk/lib/Target/X86/X86IntrinsicsInfo.h:1222
@@ -1217,1 +1221,3 @@
+  X86_INTRINSIC_DATA(avx512_mask_psra_di_512,   INTR_TYPE_2OP_MASK, X86ISD::VSRAI, 0),
   X86_INTRINSIC_DATA(avx512_mask_psra_q,        INTR_TYPE_2OP_MASK, X86ISD::VSRA, 0),
+  X86_INTRINSIC_DATA(avx512_mask_psra_q_128,    INTR_TYPE_2OP_MASK, X86ISD::VSRA, 0),
----------------
I think we should try to stay align with the _512 suffix.
maybe we should write: avx512_mask_psra_q_512, same for avx512_mask_psra_d


Repository:
  rL LLVM

http://reviews.llvm.org/D15851