[PATCH] D109907: [DSE] Use cached escape info for calls.

Alina Sbirlea via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 16 15:18:11 PDT 2021


asbirlea added a comment.

Please document in detail that callCapturesBefore should only be called if the object does not escape before the call. Even if the public API is not available with this patch, it can be added in the future due to oversight if a comment does not explain this choice.



================
Comment at: llvm/lib/Analysis/AliasAnalysis.cpp:741
+
+ModRefInfo AAResults::callKnownNoCapture(const Instruction *I,
+                                         const MemoryLocation &MemLoc,
----------------
Define the API only for a CallBase; simplify checks below.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109907



More information about the llvm-commits mailing list