<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, Apr 6, 2018 at 2:05 PM, Caio Marcelo de Oliveira Filho <span dir="ltr"><<a href="mailto:caio.oliveira@intel.com" target="_blank">caio.oliveira@intel.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 Tue, Apr 03, 2018 at 11:32:50AM -0700, Jason Ekstrand wrote:<br>
> Sometimes it's useful for a pass to be able to clean up its own derefs<br>
> instead of waiting for DCE.  This little helper makes it very easy.<br>
> ---<br>
>  src/compiler/nir/nir.h       |  2 ++<br>
>  src/compiler/nir/nir_deref.c | 13 +++++++++++++<br>
>  2 files changed, 15 insertions(+)<br>
<br>
<br>
</span>The helper is used in earlier patches, so maybe reorder. If I'm not<br>
mistaken it is used as early as patch 13 ("nir: Support deref<br>
instructions in remove_dead_variables").<br></blockquote><div><br></div><div>Yeah, I noticed this while rebasing.  The earliest use appears to be in lower_var_copies.  I've moved it to right before that patch. <br></div></div><br></div></div>