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

Ahmed Bougacha via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 1 14:47:27 PST 2016


ab accepted this revision.
ab added a comment.
This revision is now accepted and ready to land.

In http://reviews.llvm.org/D17681#365326, @RKSimon wrote:

> 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.


Sorry, you're right.

LGTM!


Repository:
  rL LLVM

http://reviews.llvm.org/D17681





More information about the llvm-commits mailing list