[PATCH] D22324: Move a transform from InstCombine to InstSimplify.

Michael Kuperstein via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 13 17:21:07 PDT 2016


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

LGTM


================
Comment at: lib/Analysis/ConstantFolding.cpp:1862
@@ +1861,3 @@
+    SmallVector<Constant *, 32> NewElements;
+    for (unsigned I = 0, E = VTy->getNumElements(); I != E; ++I) {
+      auto *MaskElt =
----------------
UndefValue* is a Constant*.
But I agree, it doesn't really matter at this point.


http://reviews.llvm.org/D22324





More information about the llvm-commits mailing list