[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