Answer to Philip Reames comments
authorRobin Morisset <morisset@google.com>
Mon, 18 Aug 2014 22:18:14 +0000 (22:18 +0000)
committerRobin Morisset <morisset@google.com>
Mon, 18 Aug 2014 22:18:14 +0000 (22:18 +0000)
commit0acd42142a8cbc7e1ca9b13df3e64e8dc727b51e
tree4a4ec20c413ae1fc20e657e09a2e988c513b37c2
parent6c0e1e0fa658f4e7466c6787aedce992ece2db55
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

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@215943 91177308-0d34-0410-b5e6-96231b3b80d8
lib/Analysis/MemoryDependenceAnalysis.cpp
test/Transforms/GVN/atomic.ll