[llvm] r183167 - Delete dead safety check.

Nick Lewycky nicholas at mxc.ca
Mon Jun 3 16:15:21 PDT 2013


Author: nicholas
Date: Mon Jun  3 18:15:20 2013
New Revision: 183167

URL: http://llvm.org/viewvc/llvm-project?rev=183167&view=rev
Log:
Delete dead safety check.

Modified:
    llvm/trunk/lib/Transforms/InstCombine/InstCombineVectorOps.cpp

Modified: llvm/trunk/lib/Transforms/InstCombine/InstCombineVectorOps.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/InstCombine/InstCombineVectorOps.cpp?rev=183167&r1=183166&r2=183167&view=diff
==============================================================================
--- llvm/trunk/lib/Transforms/InstCombine/InstCombineVectorOps.cpp (original)
+++ llvm/trunk/lib/Transforms/InstCombine/InstCombineVectorOps.cpp Mon Jun  3 18:15:20 2013
@@ -820,12 +820,7 @@ Instruction *InstCombiner::visitShuffleV
     if (isRHSID) return ReplaceInstUsesWith(SVI, RHS);
   }
 
-  if (isa<UndefValue>(RHS) &&
-      // This isn't necessary for correctness, but the comment block below
-      // claims that there are cases where folding two shuffles into one would
-      // cause worse codegen on some targets.
-      !isa<ShuffleVectorInst>(LHS) &&
-      CanEvaluateShuffled(LHS, Mask)) {
+  if (isa<UndefValue>(RHS) && CanEvaluateShuffled(LHS, Mask)) {
     Value *V = EvaluateInDifferentElementOrder(LHS, Mask);
     return ReplaceInstUsesWith(SVI, V);
   }





More information about the llvm-commits mailing list