[PATCH] D110021: [DSE] Re-enable calloc transformation with extra care (PR25892)

Dávid Bolvanský via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Sep 18 03:40:35 PDT 2021


xbolva00 added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/DeadStoreElimination.cpp:1855
+            if (MallocBB == MemsetBB)
+              return true;
+            auto *Ptr = Memset->getArgOperand(0);
----------------
hmm.. maybe rename this lambda as "shouldCreateCalloc" or "isProfitableToCreateCalloc" or something like that?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D110021



More information about the llvm-commits mailing list