[PATCH] D124124: [LangRef] Limit read/writeonly attrs to memory visible to caller

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 27 13:58:59 PDT 2022


fhahn added a comment.

In D124124#3472154 <https://reviews.llvm.org/D124124#3472154>, @efriedma wrote:

> "outside the function" doesn't really make it clear that you're talking about a dynamic invocation of the function, as opposed to instructions inside the function's body.  But maybe it's close enough.

Hmm, maybe it could be even clearer. Do you have any suggestions?  I guess we could try to spell out a definition of function-local memory somewhere and refer back to it, but I am not sure where the best place would be;


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D124124



More information about the llvm-commits mailing list