[PATCH] D116825: [GVN] MemorySSA for GVN: use MemorySSA for redundant loads elimination

[Momchil Velikov via Phabricator via llvm-commits]

chill added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/GVN.cpp:2265
+      // May happen with atomic/volatile load/store or MemorySSA imprecision.
+      if (!ClobberInst->isAtomic() ||
+          !isStrongerThan(getOrdering(ClobberInst), AtomicOrdering::Monotonic))
----------------
FIXME: This is way too conservative.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D116825/new/

https://reviews.llvm.org/D116825