[PATCH] D26902: [AVX-512] Add support for changing the element size of PALIGNR/VALIGND/VALIGNQ shuffles if they feed a vselect with a different type

Elena Demikhovsky via llvm-commits llvm-commits at lists.llvm.org
Sun Nov 20 23:44:09 PST 2016


delena accepted this revision.
delena added inline comments.
This revision is now accepted and ready to land.


================
Comment at: lib/Target/X86/X86ISelLowering.cpp:27803
+  case X86ISD::VALIGN: {
+    if (EltVT != MVT::i32 && EltVT != MVT::i64)
+      return false;
----------------
craig.topper wrote:
> delena wrote:
> > What happens with FP types? Can we do the same?
> That would result in making the VALIGN node have floating point type which we don't have patterns for today.
I think we can add FP types to VALIGN.


https://reviews.llvm.org/D26902





More information about the llvm-commits mailing list