[PATCH] D30703: [DSE] Merge stores when the later store only writes to memory locations the early store also wrote to.

Sanjoy Das via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun May 14 14:38:10 PDT 2017


sanjoy added inline comments.


================
Comment at: lib/Transforms/Scalar/DeadStoreElimination.cpp:1192
+
+            auto *SI = new StoreInst(
+                ConstantInt::get(Earlier->getValueOperand()->getType(), Merged),
----------------
You also need to copy over the atomicity of the previous store (which can only be `unordered`).


https://reviews.llvm.org/D30703





More information about the llvm-commits mailing list