[PATCH] D79391: [DSE] Remove noop stores in MSSA.

[JF Bastien via Phabricator via llvm-commits]

jfb added inline comments.


================
Comment at: llvm/test/Transforms/DeadStoreElimination/MSSA/simple.ll:588
+  store i32 %a, i32* %Q
+  ret void
+}
----------------
Can you add the reverse:
```
  %a = load i32, i32* %Q
  store volatile i32 %a, i32* %Q
```

This one we can't eliminate.

As well as, can you add these two cases with `atomic` instead? We can do some optimizations there but need to be careful around release sequences, see https://wg21.link/n4455


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79391