<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Fri, Jan 11, 2019 at 1:55 PM Kenneth Graunke <<a href="mailto:kenneth@whitecape.org">kenneth@whitecape.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Friday, January 11, 2019 8:33:41 AM PST Jason Ekstrand wrote:<br>
> I think I kind of like having "mem" be on external things.  Shared is a<br>
> little weird there because it never leaves the chip so is it mem or shader?<br>
<br>
On Intel GPUs, "shared" maps to a concept called "Shared Local Memory".<br>
So I tend to think of it as memory :)<br>
<br>
It's not perfect, though.  While most shader_in/shader_out end up being<br>
thread-local, shader_out in a TCS is actually shared across threads (and<br>
pre-dates the shared keyword).<br>
<br>
I'm sort of inclined to leave that alone for now unless you think we<br>
ought to do something about it.<br></blockquote><div><br></div><div>If by "leave it alone" you mean leave _mem on there, that's fine with me.  It was more of a philosophical question than an actual request for any sort of change.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
> > We may also want to rename the nir->globals list, or<br>
> > nir_lower_global_vars_to_local and nir_opt_global_to_local.  Not sure.<br>
> ><br>
> <br>
> Yes, whatever we do, we should make those lists more consistent.<br>
> <br>
<br>
</blockquote></div></div>