[PATCH] D17681: [X86][AVX] Better support for the variable mask form of VPERMILPD/VPERMILPS

[Simon Pilgrim via llvm-commits]

RKSimon added a comment.

In http://reviews.llvm.org/D17681#364823, @ab wrote:

> This has the same problem as VPERMV, so probably needs to wait for http://reviews.llvm.org/D17041?


I think variable mask VPERMILPS does have the 'normal' order (mask at the end) but I'm happy to wait until http://reviews.llvm.org/D17041 is dealt with.


================
Comment at: lib/Target/X86/X86IntrinsicsInfo.h:332
@@ +331,3 @@
+  X86_INTRINSIC_DATA(avx_vpermilvar_pd, INTR_TYPE_2OP, X86ISD::VPERMILPV, 0),
+  X86_INTRINSIC_DATA(avx_vpermilvar_pd_256, INTR_TYPE_2OP, X86ISD::VPERMILPV, 0),
+  X86_INTRINSIC_DATA(avx_vpermilvar_ps, INTR_TYPE_2OP, X86ISD::VPERMILPV, 0),
----------------
ab wrote:
> Maybe align the columns?
No problem.


Repository:
  rL LLVM

http://reviews.llvm.org/D17681