aboutsummaryrefslogtreecommitdiff
path: root/llvm/lib/Transforms/Utils/SimplifyCFG.cpp
diff options
context:
space:
mode:
authorJames Molloy <james.molloy@arm.com>2016-08-31 13:16:36 +0000
committerJames Molloy <james.molloy@arm.com>2016-08-31 13:16:36 +0000
commit06a45483a179d125c8c76419c70fc1f173cfb698 (patch)
treeaadbd31638da0b05e7dad9d3a59c65b439574caf /llvm/lib/Transforms/Utils/SimplifyCFG.cpp
parent8a66a39cbf6d8b514f126c4098a7e9e72b5a0ffd (diff)
downloadllvm-06a45483a179d125c8c76419c70fc1f173cfb698.zip
llvm-06a45483a179d125c8c76419c70fc1f173cfb698.tar.gz
llvm-06a45483a179d125c8c76419c70fc1f173cfb698.tar.bz2
Revert "[SimplifyCFG] Add a workaround to fix PR30188"
This reverts commit r280219. r280216 caused buildbot failures - backing out the entire chain. llvm-svn: 280232
Diffstat (limited to 'llvm/lib/Transforms/Utils/SimplifyCFG.cpp')
-rw-r--r--llvm/lib/Transforms/Utils/SimplifyCFG.cpp10
1 files changed, 0 insertions, 10 deletions
diff --git a/llvm/lib/Transforms/Utils/SimplifyCFG.cpp b/llvm/lib/Transforms/Utils/SimplifyCFG.cpp
index 3de6c60..3404a4d 100644
--- a/llvm/lib/Transforms/Utils/SimplifyCFG.cpp
+++ b/llvm/lib/Transforms/Utils/SimplifyCFG.cpp
@@ -1424,16 +1424,6 @@ static bool canSinkInstructions(
// FIXME: if the call was *already* indirect, we should do this.
return false;
}
- // Because SROA can't handle speculating stores of selects, try not
- // to sink stores of allocas when we'd have to create a PHI for the
- // address operand.
- // FIXME: This is a workaround for a deficiency in SROA - see
- // https://llvm.org/bugs/show_bug.cgi?id=30188
- if (OI == 1 && isa<StoreInst>(I0) &&
- any_of(Insts, [](const Instruction *I) {
- return isa<AllocaInst>(I->getOperand(1));
- }))
- return false;
for (auto *I : Insts)
PHIOperands[I].push_back(I->getOperand(OI));
}