<div dir="auto"><div><br><div class="gmail_extra"><br><div class="gmail_quote">On Dec 27, 2016 11:04 AM, "Jordan Justen" <<a href="mailto:jordan.l.justen@intel.com">jordan.l.justen@intel.com</a>> wrote:<br type="attribution"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Series Reviewed-by: Jordan Justen <<a href="mailto:jordan.l.justen@intel.com">jordan.l.justen@intel.com</a>></blockquote></div></div></div><div dir="auto"><br></div><div dir="auto">Thanks!</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_extra"><div class="gmail_quote"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="elided-text">
On 2016-12-26 09:13:27, Jason Ekstrand wrote:<br>
> The first patch in this series is a change I've been meaning to make for<br>
> some time.  I've been running into name collisions with nir_copy_deref and<br>
> the nir_builder nir_copy_deref_var helper.  Also, now that we have lots of<br>
> other "clone" stuff, it makes sense to make things make them all match.<br>
><br>
> The other 7 patches are fixes that update some passes a bit and make us<br>
> handle ralloc memory contexts better.<br>
><br>
> Jason Ekstrand (8):<br>
>   nir: Make nir_copy_deref follow the "clone" pattern<br>
>   nir/from_ssa: Use nir_builder for emit_copy<br>
>   nir/lower_io: Use the builder instead of carrying a mem_ctx<br>
>   nir/lower_var_copies: Use a shader rather than a void *mem_ctx<br>
>   nir/conditional_if: Properly use the builder<br>
>   nir/opt_peephole_select: Pass around the actual nir_shader<br>
>   nir/split_var_copies: Use a nir_shader rather than a void *mem_ctx<br>
>   i965/peephole_ffma: Use nir_builder<br>
><br>
>  src/compiler/glsl/glsl_to_nir.<wbr>cpp                  |  3 +-<br>
>  src/compiler/nir/nir.c                             | 22 +++++------<br>
>  src/compiler/nir/nir.h                             |  6 +--<br>
>  src/compiler/nir/nir_builder.h                     |  6 +--<br>
>  src/compiler/nir/nir_from_ssa.<wbr>c                    | 26 ++++++-------<br>
>  src/compiler/nir/nir_inline_<wbr>functions.c            |  2 +-<br>
>  src/compiler/nir/nir_lower_<wbr>indirect_derefs.c       |  6 +--<br>
>  src/compiler/nir/nir_lower_io.<wbr>c                    | 16 ++++----<br>
>  src/compiler/nir/nir_lower_<wbr>tex.c                   | 18 +++------<br>
>  src/compiler/nir/nir_lower_<wbr>var_copies.c            | 21 ++++++-----<br>
>  src/compiler/nir/nir_opt_<wbr>conditional_discard.c     | 21 +++++------<br>
>  src/compiler/nir/nir_opt_<wbr>peephole_select.c         |  9 +++--<br>
>  src/compiler/nir/nir_split_<wbr>var_copies.c            | 23 +++++-------<br>
>  src/compiler/spirv/spirv_to_<wbr>nir.c                  | 22 +++++------<br>
>  src/compiler/spirv/vtn_<wbr>glsl450.c                   |  3 +-<br>
>  src/compiler/spirv/vtn_<wbr>variables.c                 |  3 +-<br>
>  src/intel/vulkan/anv_nir_<wbr>lower_input_attachments.c |  3 +-<br>
>  .../drivers/dri/i965/brw_nir_<wbr>opt_peephole_ffma.c   | 43 +++++++---------------<br>
>  18 files changed, 111 insertions(+), 142 deletions(-)<br>
><br>
> --<br>
> 2.5.0.400.gff86faf<br>
><br>
</div>> ______________________________<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>