diff options
| author | Wojciech Matyjewicz <wmatyjewicz@fastmail.fm> | 2008-06-15 19:07:39 +0000 |
|---|---|---|
| committer | Wojciech Matyjewicz <wmatyjewicz@fastmail.fm> | 2008-06-15 19:07:39 +0000 |
| commit | 386e26b0e33c6e142e2a456ae670ddc1c2e168a1 (patch) | |
| tree | 9abb039b6c4ffddfb82d5c4ad11145c2880bbbd7 /test | |
| parent | ce6f137a01884d87c77f34ec7845d0b36222e2e3 (diff) | |
| download | external_llvm-386e26b0e33c6e142e2a456ae670ddc1c2e168a1.zip external_llvm-386e26b0e33c6e142e2a456ae670ddc1c2e168a1.tar.gz external_llvm-386e26b0e33c6e142e2a456ae670ddc1c2e168a1.tar.bz2 | |
Fix PR2434. When scanning for exising binary operator to reuse don't
take into account the instrucion pointed by InsertPt. Thanks to it,
returning the new value of InsertPt to the InsertBinop() caller can be
avoided. The bug was, actually, in visitAddRecExpr() method which wasn't
correctly handling changes of InsertPt. There shouldn't be any
performance regression, as -gvn pass (run after -indvars) removes any
redundant binops.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@52291 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'test')
| -rw-r--r-- | test/Transforms/IndVarsSimplify/2008-06-15-SCEVExpanderBug.ll | 17 |
1 files changed, 17 insertions, 0 deletions
diff --git a/test/Transforms/IndVarsSimplify/2008-06-15-SCEVExpanderBug.ll b/test/Transforms/IndVarsSimplify/2008-06-15-SCEVExpanderBug.ll new file mode 100644 index 0000000..aac8d97 --- /dev/null +++ b/test/Transforms/IndVarsSimplify/2008-06-15-SCEVExpanderBug.ll @@ -0,0 +1,17 @@ +; RUN: llvm-as < %s | opt -indvars -disable-output +; PR2434 + +define fastcc void @regcppop() nounwind { +entry: + %tmp61 = add i32 0, -5 ; <i32> [#uses=1] + br label %bb + +bb: ; preds = %bb, %entry + %PL_savestack_ix.tmp.0 = phi i32 [ %tmp61, %entry ], [ %tmp127, %bb ] ; <i32> [#uses=2] + %indvar10 = phi i32 [ 0, %entry ], [ %indvar.next11, %bb ] ; <i32> [#uses=2] + %tmp13 = mul i32 %indvar10, -4 ; <i32> [#uses=0] + %tmp111 = add i32 %PL_savestack_ix.tmp.0, -3 ; <i32> [#uses=0] + %tmp127 = add i32 %PL_savestack_ix.tmp.0, -4 ; <i32> [#uses=1] + %indvar.next11 = add i32 %indvar10, 1 ; <i32> [#uses=1] + br label %bb +} |