[PATCH] D97924: [LangRef] clarify the semantics of nocapture
Nuno Lopes via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Apr 12 11:25:37 PDT 2021
nlopes added a comment.
@aqjune ping: there are a few sentences that need tweaking.
================
Comment at: llvm/docs/LangRef.rst:2642
+results of operations such as :ref:`ptrtoint <i_ptrtoint>` and thus alters the
+final state.
+
----------------
jdoerfert wrote:
> I don't really like this text too much. Storing is only one way and I'm not sure why we need to talk about non-integral pointers at all.
>
> @reames and me recently started a write up to define "capture", maybe we should share that.
agreed. this non-integral pointer text isn't very readable to me either.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D97924/new/
https://reviews.llvm.org/D97924
More information about the llvm-commits
mailing list