[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