[PATCH] D45535: [DSE] Teach the pass that atomic memory intrinsics are stores.

[Dave Green via Phabricator via llvm-commits]

dmgreen added a comment.

Am I right in saying these are treated in the same way as atomic stores are? In that they can be deleted by following non-atomic stores, even if they are "stronger" atomically?  If so, this looks fine to me.



================
Comment at: lib/Transforms/Scalar/DeadStoreElimination.cpp:868
       LoadedLoc = MemoryLocation::get(V);
     } else if (MemTransferInst *MTI = dyn_cast<MemTransferInst>(BBI)) {
       LoadedLoc = MemoryLocation::getForSource(MTI);
----------------
This one? This function does it's own special version of DSE. It's about removing stores to dead stack objects iirc.


Repository:
  rL LLVM

https://reviews.llvm.org/D45535