diff options
| author | Robin Morisset <morisset@google.com> | 2014-08-18 22:18:14 +0000 |
|---|---|---|
| committer | Robin Morisset <morisset@google.com> | 2014-08-18 22:18:14 +0000 |
| commit | 9e98e7f7fc5b06db5c8fc6dc872689e7ef9b9888 (patch) | |
| tree | 7bd6530ee71e5eefdf3b48567a16c4e576edcdde /llvm/lib/ExecutionEngine/ExecutionEngine.cpp | |
| parent | 4ffe8aaa694acceb58c01ffea7c1684b3d80e2fe (diff) | |
| download | llvm-9e98e7f7fc5b06db5c8fc6dc872689e7ef9b9888.zip llvm-9e98e7f7fc5b06db5c8fc6dc872689e7ef9b9888.tar.gz llvm-9e98e7f7fc5b06db5c8fc6dc872689e7ef9b9888.tar.bz2 | |
Answer to Philip Reames comments
- add check for volatile (probably unneeded, but I agree that we should be conservative about it).
- strengthen condition from isUnordered() to isSimple(), as I don't understand well enough Unordered semantics (and it also matches the comment better this way) to be confident in the previous behaviour (thanks for catching that one, I had missed the case Monotonic/Unordered).
- separate a condition in two.
- lengthen comment about aliasing and loads
- add tests in GVN/atomic.ll
llvm-svn: 215943
Diffstat (limited to 'llvm/lib/ExecutionEngine/ExecutionEngine.cpp')
0 files changed, 0 insertions, 0 deletions