diff options
| author | Bill Wendling <isanbard@gmail.com> | 2011-08-15 21:14:31 +0000 |
|---|---|---|
| committer | Bill Wendling <isanbard@gmail.com> | 2011-08-15 21:14:31 +0000 |
| commit | e86965ee19943aa5b4dc4b4cabad2a0c25857f77 (patch) | |
| tree | 58e7cce41ad3a8f319298eb4d2e00780b978fbde /llvm/lib/Transforms | |
| parent | 4d05198d1f3f792a61641627c3abbb1158acea8c (diff) | |
| download | bcm5719-llvm-e86965ee19943aa5b4dc4b4cabad2a0c25857f77.tar.gz bcm5719-llvm-e86965ee19943aa5b4dc4b4cabad2a0c25857f77.zip | |
Duncan pointed out that the LandingPadInst might read memory. (It might also
write to memory.) Marking it as such makes some checks for immobility go away.
llvm-svn: 137655
Diffstat (limited to 'llvm/lib/Transforms')
| -rw-r--r-- | llvm/lib/Transforms/InstCombine/InstructionCombining.cpp | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/llvm/lib/Transforms/InstCombine/InstructionCombining.cpp b/llvm/lib/Transforms/InstCombine/InstructionCombining.cpp index 32c55a3408d..41d542af685 100644 --- a/llvm/lib/Transforms/InstCombine/InstructionCombining.cpp +++ b/llvm/lib/Transforms/InstCombine/InstructionCombining.cpp @@ -1417,9 +1417,8 @@ Instruction *InstCombiner::visitExtractValueInst(ExtractValueInst &EV) { static bool TryToSinkInstruction(Instruction *I, BasicBlock *DestBlock) { assert(I->hasOneUse() && "Invariants didn't hold!"); - // Cannot move control-flow-involving, volatile loads, vaarg, landingpad, etc. - if (isa<PHINode>(I) || isa<LandingPadInst>(I) || I->mayHaveSideEffects() || - isa<TerminatorInst>(I)) + // Cannot move control-flow-involving, volatile loads, vaarg, etc. + if (isa<PHINode>(I) || I->mayHaveSideEffects() || isa<TerminatorInst>(I)) return false; // Do not sink alloca instructions out of the entry block. |
