[PATCH] D100141: [nofree] Restrict semantics to memory visible to caller

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 16 11:32:32 PDT 2021


reames added a comment.

In D100141#2695406 <https://reviews.llvm.org/D100141#2695406>, @jdoerfert wrote:

> LGTM, @nhaehnle should comment on the new wording probably.

JFYI, I don't intend to hold for @nhaehnle as the concern he raised appears orthogonal to the change being made in this review.  I am happy to continue this discussion and post another patch if we thing further clarification is warranted for the concurrency case.  I am leaning in that direction myself.

Doing a final rebuild now, and will submit after that.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100141



More information about the llvm-commits mailing list