[PATCH] D18094: [x86, AVX] replace masked load with full vector load when possible

Simon Pilgrim via llvm-commits llvm-commits at lists.llvm.org
Sun Mar 13 13:57:16 PDT 2016


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

LGTM with one minor comment


================
Comment at: lib/Target/X86/X86ISelLowering.cpp:27371
@@ -27349,4 +27370,3 @@
   // cause an infinite loop because that's what we're about to create.
-  if (!ISD::isBuildVectorOfConstantSDNodes(ML->getMask().getNode()) ||
-      ML->getSrc0().getOpcode() == ISD::UNDEF)
+  if (ML->getSrc0().getOpcode() == ISD::UNDEF)
     return SDValue();
----------------
if (ML->getSrc0().isUndef())


http://reviews.llvm.org/D18094





More information about the llvm-commits mailing list