[PATCH] D132657: [DSE] Eliminate noop store even through has clobbering between LoadI and StoreI

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 31 00:00:49 PDT 2022


nikic accepted this revision.
nikic added a comment.
This revision is now accepted and ready to land.

LG



================
Comment at: llvm/lib/Transforms/Scalar/DeadStoreElimination.cpp:1868
+            if (CurrentStoreI && CurrentStoreI->getOperand(0) == LoadI) {
+              // This is a potentially clobbering here, but it writes the same value,
+              // so we can safely ignore it.
----------------
here -> store?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D132657



More information about the llvm-commits mailing list