[PATCH] D121415: [FunctionAttrs] Infer argmemonly .
Florian Hahn via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Mar 15 08:11:50 PDT 2022
fhahn marked 2 inline comments as done.
fhahn added inline comments.
================
Comment at: llvm/lib/Transforms/IPO/FunctionAttrs.cpp:145
+ const Value *UO = getUnderlyingObject(Ptr);
+ return UO && (isa<Argument>(UO) || isa<AllocaInst>(UO));
+ };
----------------
nikic wrote:
> getUnderlyingObject() never returns null.
Removed the extra check, thanks!
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D121415/new/
https://reviews.llvm.org/D121415
More information about the llvm-commits
mailing list