[PATCH] D117005: [DSE] Remove alloc function check in canSkipDef()

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Jan 16 08:33:04 PST 2022


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

LGTM, thanks!



================
Comment at: llvm/lib/Transforms/Scalar/DeadStoreElimination.cpp:703
 bool canSkipDef(MemoryDef *D, bool DefVisibleToCaller,
                 const TargetLibraryInfo &TLI) {
   Instruction *DI = D->getMemoryInst();
----------------
as @yurai007 suggested, it looks like the TLI arg can be dropped


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

https://reviews.llvm.org/D117005



More information about the llvm-commits mailing list