<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Fri, Jan 11, 2019 at 11:11 AM 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>
> On Fri, Jan 11, 2019 at 10:19 AM Kenneth Graunke wrote:<br>
> > Those names (nir_var_func_local, nir_var_thread_local, and<br>
> > nir_var_thread_global) make more sense to me than private/function.<br>
> ><br>
> > Another option is `nir_var_local_temp` and `nir_var_shader_temp`,<br>
> > indicating that they're just temporary variables, and not anything<br>
> > with special semantics like memory.  shader_temp would pair well with<br>
> > the existing shader_in/shader_out, since they have the same scope.<br>
> ><br>
> > I might also consider adding 'mem' to variables representing memory.<br>
> ><br>
> > So that would look like...<br>
> ><br>
> >    nir_var_shader_in<br>
> >    nir_var_shader_out<br>
> >    nir_var_shader_temp  (formerly local/function)<br>
> >    nir_var_local_temp   (formerly global/private)<br>
> ><br>
> <br>
> Are those flipped?<br>
<br>
Gah!  Sorry.  Yes.<br>
<br>
   nir_var_shader_in<br>
   nir_var_shader_out<br>
   nir_var_shader_temp  (formerly global/private)<br>
   nir_var_local_temp   (formerly local/function)<br>
   nir_var_uniform<br>
   nir_var_system_value<br>
   nir_var_mem_ubo      (added mem)<br>
   nir_var_mem_ssbo     (added mem)<br>
   nir_var_mem_shared   (added mem)<br>
   nir_var_mem_global   (the new global memory type being introduced)<br></blockquote><div><br></div><div>I can work with that.  I do think I'd mildly prefer function_temp over local_temp but I think the adding of _temp is an improvement. <br></div></div></div>