diff options
| author | Chris Lattner <sabre@nondot.org> | 2008-12-18 00:51:32 +0000 |
|---|---|---|
| committer | Chris Lattner <sabre@nondot.org> | 2008-12-18 00:51:32 +0000 |
| commit | 5f4f84ba3ec3cd5c9d6fb8acd770042a5cfdf8aa (patch) | |
| tree | 5dd02ea47fad3d5d4f9008636ff7fcd44d2c6305 | |
| parent | a5d5c0268ad4f95c71e76b8a05118292dd12daa5 (diff) | |
reapply this hunk from Bill's reversion in r61169, it is conservative
and safe and orthogonal from turning off load pre.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@61177 91177308-0d34-0410-b5e6-96231b3b80d8
| -rw-r--r-- | lib/Transforms/Scalar/GVN.cpp | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/lib/Transforms/Scalar/GVN.cpp b/lib/Transforms/Scalar/GVN.cpp index 3662424923..6e532630c7 100644 --- a/lib/Transforms/Scalar/GVN.cpp +++ b/lib/Transforms/Scalar/GVN.cpp @@ -956,6 +956,11 @@ bool GVN::processNonLocalLoad(LoadInst *LI, // it will be too expensive. if (Deps.size() > 100) return false; + + // If we had a phi translation failure, we'll have a single entry which is a + // clobber in the current block. Reject this early. + if (Deps.size() == 1 && Deps[0].second.isClobber()) + return false; // Filter out useless results (non-locals, etc). Keep track of the blocks // where we have a value available in repl, also keep track of whether we see |