[PATCH] D84432: [IPSCCP] Drop argmemonly after replacing pointer argument.

[Florian Hahn via Phabricator via llvm-commits]

fhahn added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/SCCP.cpp:1927
+        F.removeFnAttr(Attribute::ArgMemOnly);
+        F.removeFnAttr(Attribute::InaccessibleMemOrArgMemOnly);
+      }
----------------
efriedma wrote:
> Do we also need to modify the attributes on the callsites?
Yes, updated!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84432