[PATCH] D114872: [MemoryLocation] Move DSE's logic to new MemLoc::getForDest helper (NFC).

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 1 07:04:11 PST 2021


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

LG



================
Comment at: llvm/lib/Analysis/MemoryLocation.cpp:149
+
+  return {};
+}
----------------
nit: `return None;`


================
Comment at: llvm/lib/Transforms/Scalar/DeadStoreElimination.cpp:1039
       case Intrinsic::masked_store:
         return {MemoryLocation::getForArgument(CB, 1, TLI)};
       default:
----------------
Move these two over as well?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D114872



More information about the llvm-commits mailing list