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

Zoe Carver via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 4 21:32:17 PDT 2020


zoecarver marked an inline comment as done.
zoecarver added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/DeadStoreElimination.cpp:1814
+    // Check if we're storing a value that we just loaded.
+    if (auto *load = dyn_cast<LoadInst>(SI->getOperand(0))) {
+      if (load->getPointerOperand() == SI->getOperand(1)) {
----------------
Forgot about the naming convention, my bad. Will fix. 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79391





More information about the llvm-commits mailing list