diff options
| author | Eli Friedman <eli.friedman@gmail.com> | 2011-09-12 20:23:13 +0000 |
|---|---|---|
| committer | Eli Friedman <eli.friedman@gmail.com> | 2011-09-12 20:23:13 +0000 |
| commit | 2bc3d52b9ab422ee9f7e42a1a4e3b818e623a5f7 (patch) | |
| tree | 399efba22c6e8847e43d3bd5f8561c8a194b6aa6 /lib/Transforms/Scalar/JumpThreading.cpp | |
| parent | a0737950233acc271e444b270498d4986b4d2d00 (diff) | |
| download | llvm-2bc3d52b9ab422ee9f7e42a1a4e3b818e623a5f7.tar.gz llvm-2bc3d52b9ab422ee9f7e42a1a4e3b818e623a5f7.tar.bz2 llvm-2bc3d52b9ab422ee9f7e42a1a4e3b818e623a5f7.tar.xz | |
Change a bunch of isVolatile() checks to check for atomic load/store as well.
No tests; these changes aren't really interesting in the sense that the logic is the same for volatile and atomic.
I believe this completes all of the changes necessary for the optimizer to handle loads and stores correctly. I'm going to try and come up with some additional testing, though.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@139533 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Transforms/Scalar/JumpThreading.cpp')
| -rw-r--r-- | lib/Transforms/Scalar/JumpThreading.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/lib/Transforms/Scalar/JumpThreading.cpp b/lib/Transforms/Scalar/JumpThreading.cpp index b500d5b4fd..f410af3275 100644 --- a/lib/Transforms/Scalar/JumpThreading.cpp +++ b/lib/Transforms/Scalar/JumpThreading.cpp @@ -811,8 +811,8 @@ bool JumpThreading::ProcessBlock(BasicBlock *BB) { /// important optimization that encourages jump threading, and needs to be run /// interlaced with other jump threading tasks. bool JumpThreading::SimplifyPartiallyRedundantLoad(LoadInst *LI) { - // Don't hack volatile loads. - if (LI->isVolatile()) return false; + // Don't hack volatile/atomic loads. + if (!LI->isSimple()) return false; // If the load is defined in a block with exactly one predecessor, it can't be // partially redundant. |