<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jan 9, 2015 at 10:58 AM, Connor Abbott <span dir="ltr"><<a href="mailto:cwabbott0@gmail.com" target="_blank">cwabbott0@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Sun, Jan 4, 2015 at 4:01 PM, Connor Abbott <<a href="mailto:cwabbott0@gmail.com">cwabbott0@gmail.com</a>> wrote:<br>
</span>[...]<br>
<span class="">>> static bool<br>
>> deref_may_be_aliased(nir_deref_var *deref,<br>
>>                      struct lower_variables_state *state)<br>
>> {<br>
>>    nir_deref_var var_deref = *deref;<br>
>>    var_deref.deref.child = NULL;<br>
>>    struct deref_node *node = get_deref_node(&var_deref, false, state);<br>
><br>
> We're only looking for the root node corresponding to the variable, and<br>
> we've already populated the deref_var_nodes table, so we can just look it up<br>
> here.<br>
><br>
<br>
</span>I think you missed this, since the code is still the same and you<br>
never responded... it seems strange to me to call this complicated<br>
function (get_deref_node()) when all you want to do is look up<br>
something in a hash table.<br>
</blockquote></div><br></div><div class="gmail_extra">I did that so that we keep all the lookup code for deref nodes in one place.  Also, it ensures that the node gets create if it isn't already.  That said, it should be already so that shouldn't matter.  In any case, I just sent a cleanup patch that should fix it for you.<br></div></div>