<div dir="ltr"><div><div><div>1-27/42 and 29-42/42 are<br><br></div>Reviewed-by: Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>><br><br></div>28/42 will take more brain power than I can exert tonight.  I'll look at it tomorrow.<br><br></div>Thanks for working on this.  I know how much pain refactoring can be.<br><div><div><div><div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Oct 19, 2016 at 4:09 PM, Timothy Arceri <span dir="ltr"><<a href="mailto:timothy.arceri@collabora.com" target="_blank">timothy.arceri@collabora.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">V3:<br>
- create a temporary copy_shader_info() helper and split up the changes<br>
  in this series for easier review as suggested by Jason/Ken.<br>
- a couple of small changes as suggested by Ian and Marek (more details in<br>
  patches).<br>
<br>
Having the shader info struct in nir_shader makes things difficult for<br>
the shader cache, and means we are copying values in the glsl->nir pass<br>
unnecessarily. This series creates a shared shader info and makes a bunch<br>
of clean-ups based on this change.<br>
<br>
The st changes have been tested against llvmpipe.<br>
<br>
There is still more work to be done to remove some of the now duplicate<br>
fields in gl_program but I'll send that in a follow up series.<br>
<br>
______________________________<wbr>_________________<br>
mesa-dev mailing list<br>
<a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a><br>
<a href="https://lists.freedesktop.org/mailman/listinfo/mesa-dev" rel="noreferrer" target="_blank">https://lists.freedesktop.org/<wbr>mailman/listinfo/mesa-dev</a><br>
</blockquote></div><br></div></div></div></div></div></div>