[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