diff options
| author | James Molloy <james.molloy@arm.com> | 2016-08-01 09:34:48 +0000 |
|---|---|---|
| committer | James Molloy <james.molloy@arm.com> | 2016-08-01 09:34:48 +0000 |
| commit | bade86cedcba12c21677b2ccf0ed6ada2f3ef268 (patch) | |
| tree | 0647e20343cbcf691bc19288d4c4cba9c40748c3 | |
| parent | ab5a4c7dbb0a16ea9bc4bfc5d5bdd67798e87958 (diff) | |
| download | bcm5719-llvm-bade86cedcba12c21677b2ccf0ed6ada2f3ef268.tar.gz bcm5719-llvm-bade86cedcba12c21677b2ccf0ed6ada2f3ef268.zip | |
[SimplifyCFG] Fix nasty RAUW bug from r277325
Using RAUW was wrong here; if we have a switch transform such as:
18 -> 6 then
6 -> 0
If we use RAUW, while performing the second transform the *transformed* 6
from the first will be also replaced, so we end up with:
18 -> 0
6 -> 0
Found by clang stage2 bootstrap; testcase added.
llvm-svn: 277332
| -rw-r--r-- | llvm/lib/Transforms/Utils/SimplifyCFG.cpp | 7 | ||||
| -rw-r--r-- | llvm/test/Transforms/SimplifyCFG/rangereduce.ll | 28 |
2 files changed, 31 insertions, 4 deletions
diff --git a/llvm/lib/Transforms/Utils/SimplifyCFG.cpp b/llvm/lib/Transforms/Utils/SimplifyCFG.cpp index f4d7f5e38b9..8e821d9b9fa 100644 --- a/llvm/lib/Transforms/Utils/SimplifyCFG.cpp +++ b/llvm/lib/Transforms/Utils/SimplifyCFG.cpp @@ -5132,11 +5132,12 @@ static bool ReduceSwitchRange(SwitchInst *SI, IRBuilder<> &Builder, Builder.CreateShl(Sub, Ty->getBitWidth() - Shift)); SI->replaceUsesOfWith(SI->getCondition(), Rot); - for (auto &C : SI->cases()) { + for (SwitchInst::CaseIt C = SI->case_begin(), E = SI->case_end(); C != E; + ++C) { auto *Orig = C.getCaseValue(); auto Sub = Orig->getValue() - APInt(Ty->getBitWidth(), Base); - SI->replaceUsesOfWith(Orig, - ConstantInt::get(Ty, Sub.lshr(ShiftC->getValue()))); + C.setValue( + cast<ConstantInt>(ConstantInt::get(Ty, Sub.lshr(ShiftC->getValue())))); } return true; } diff --git a/llvm/test/Transforms/SimplifyCFG/rangereduce.ll b/llvm/test/Transforms/SimplifyCFG/rangereduce.ll index d1a745ecfd9..1b6434a4129 100644 --- a/llvm/test/Transforms/SimplifyCFG/rangereduce.ll +++ b/llvm/test/Transforms/SimplifyCFG/rangereduce.ll @@ -192,4 +192,30 @@ two: ret i32 1143 three: ret i32 99783 -}
\ No newline at end of file +} + +; CHECK-LABEL: @test9 +; CHECK: switch +; CHECK: i32 6 +; CHECK: i32 7 +; CHECK: i32 0 +; CHECK: i32 2 +define i32 @test9(i32 %a) { + switch i32 %a, label %def [ + i32 18, label %one + i32 20, label %two + i32 6, label %three + i32 10, label %three + ] + +def: + ret i32 8867 + +one: + ret i32 11984 +two: + ret i32 1143 +three: + ret i32 99783 +} + |

