[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