[PATCH] D121415: [FunctionAttrs] Infer argmemonly .

[Florian Hahn via Phabricator via llvm-commits]

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