<div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr">Hey Johannes,</div><div dir="ltr"><br></div><div dir="ltr">From my recollection of past conversations with you, in the example with `foo`, shouldn't there be a `pthread_join` call also represented in the documentation, so that the lifetime of the child call is shown to not escape the scope of the parent stack frame?<br></div><div dir="ltr"><br></div><div dir="ltr">For the !nocapture/nocapture_use optimizations, what is the intended semantics in the presence of interprocedural analysis, such as inlining? It sounds like the !nocapture is tied to the lifetime of the alloca, but the last use of the value loaded is always, necessarily, going to come after the last use of the alloca (to load the value), and thus seems insufficient to cover the lifetime of the uses of that stored pointer. </div></div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Jan 7, 2021 at 7:21 PM Johannes Doerfert via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org">llvm-dev@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">TL;DR: A pointer stored in memory is not necessarily captured, let's add <br>
a way to express this.<br>
<br>
Phab: <a href="https://reviews.llvm.org/D93189" rel="noreferrer" target="_blank">https://reviews.llvm.org/D93189</a><br>
<br>
--- Commit Message / Rational ---<br>
<br>
Runtime functions, as well as regular functions, might require a pointer<br>
to be passed in memory even though the memory is simply a means to pass<br>
(multiple) arguments. That is, the indirection through memory is only<br>
used on the call edge and not otherwise relevant. However, such pointers<br>
are currently assumed to escape as soon as they are stored in memory<br>
even if the callee only reloads them and use them in a "non-escaping" way.<br>
Generally, storing a pointer might not cause it to escape if all "uses of<br>
  the memory" it is stored to all have the "nocapture" property.<br>
<br>
To allow optimizations in the presence of pointers stored to memory we<br>
introduce two new IR extensions. `!nocapture` metadata on stores and<br>
"nocapture_use" operand bundles for call(base) instructions. The former<br>
ensures that the store can be ignored for the purpose of escape<br>
analysis. The latter indicates that a call is using a pointer value<br>
but not capturing it. This is important as the call might still read<br>
or write the pointer and since the passing of the pointer through<br>
memory is not considered "capturing" with the "nocapture" metadata,<br>
we need to otherwise indicate the potential read/write.<br>
<br>
As an example use case where we can deduce `!nocapture` metadata,<br>
consider the following code:<br>
<br>
```<br>
struct Payload {<br>
   int *a;<br>
   double *b;<br>
};<br>
<br>
int pthread_create(pthread_t *thread, const pthread_attr_t *attr,<br>
                     void *(*start_routine) (void *), void *arg);<br>
<br>
int use(double);<br>
<br>
void fn(void *v) {<br>
   Payload *p = (Payload*)(v);<br>
   // Load the pointers from the payload and then dereference them,<br>
   // this will not capture the pointers.<br>
   int *a = p->a;<br>
   double *b = p->b;<br>
   *a = use(*b);<br>
}<br>
<br>
void foo(int *a, double *b) {<br>
   Payload p = {a, b};<br>
   pthread_create(..., &fn, &p);<br>
}<br>
```<br>
<br>
Given the usage of the payload struct in `fn` we can conclude neither<br>
`a` nor `b` in are captured in `foo`, however we could not express this<br>
fact "locally" before. That is, we can deduce and annotate it for the<br>
arguments `a` and `b` but only since there is no other use (later on).<br>
Similarly, if the callee would not be known, we were not able to<br>
describe the "nocapture" behavior of the API.<br>
<br>
A follow up patch will introduce `!nocapture` metadata to stores<br>
generated during OpenMP lowering. This will, among other things, fix<br>
PR48475. I generally expect us to find more APIs that could benefit from<br>
the annotation in addition to the deduction we can do if we see the callee.<br>
<br>
---<br>
<br>
As always, feedback is welcome. Feel free to look at the phab patch as well.<br>
<br>
Thanks,<br>
   Johannes<br>
<br>
<br>
-- <br>
──────────<br>
∽ Johannes<br>
<br>
_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br>
</blockquote></div>