diff options
| author | Owen Anderson <resistor@mac.com> | 2008-09-03 23:06:07 +0000 |
|---|---|---|
| committer | Owen Anderson <resistor@mac.com> | 2008-09-03 23:06:07 +0000 |
| commit | 0c7f91cf843247a34ae9147af0708d238d43f9f9 (patch) | |
| tree | e65afd0a375b4d82e1d09660765bd2c3f4f65c00 /lib/Transforms | |
| parent | 72ab076e671bee289e9e83f23b382131ea5c1560 (diff) | |
| download | llvm-0c7f91cf843247a34ae9147af0708d238d43f9f9.tar.gz llvm-0c7f91cf843247a34ae9147af0708d238d43f9f9.tar.bz2 llvm-0c7f91cf843247a34ae9147af0708d238d43f9f9.tar.xz | |
Fix a bug that prevented PRE from applying in some cases.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@55744 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Transforms')
| -rw-r--r-- | lib/Transforms/Scalar/GVN.cpp | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/lib/Transforms/Scalar/GVN.cpp b/lib/Transforms/Scalar/GVN.cpp index 42fbc78671..b258fcc08c 100644 --- a/lib/Transforms/Scalar/GVN.cpp +++ b/lib/Transforms/Scalar/GVN.cpp @@ -1131,9 +1131,11 @@ bool GVN::runOnFunction(Function& F) { } if (EnablePRE) { - bool PREChanged = false; - while ((PREChanged = performPRE(F))) + bool PREChanged = true; + while (PREChanged) { + PREChanged = performPRE(F); changed |= PREChanged; + } } return changed; @@ -1256,7 +1258,7 @@ bool GVN::performPRE(Function& F) { unsigned succNum = 0; for (unsigned i = 0, e = PREPred->getTerminator()->getNumSuccessors(); i != e; ++i) - if (PREPred->getTerminator()->getSuccessor(i) == PREPred) { + if (PREPred->getTerminator()->getSuccessor(i) == CurrentBlock) { succNum = i; break; } |