<div dir="ltr">Awesome!<div><br></div><div>Reviewed-by: Bas Nieuwenhuizen <<a href="mailto:bas@basnieuwenhuizen.nl">bas@basnieuwenhuizen.nl</a>></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Feb 9, 2018 at 1:54 PM, Samuel Pitoiset <span dir="ltr"><<a href="mailto:samuel.pitoiset@gmail.com" target="_blank">samuel.pitoiset@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<br>
This series removes the backlink to nir_to_llvm_context in ac_nir_context<br>
by moving some code to the ABI.<br>
<br>
We should also rename nir_to_llvm_context to radv_shader_context, move<br>
all RADV-specific code outside of ac and remove the link to ac_nir_context,<br>
but this series is a good start.<br>
<br>
Please review,<br>
Thanks!<br>
<br>
Samuel Pitoiset (18):<br>
  ac/nir: remove unused nir_to_llvm_context:{defs,<wbr>phis}<br>
  ac: move tg_size to the ABI<br>
  ac: move push_constants to the ABI<br>
  ac: move view_index to the ABI<br>
  ac: add load_sample_mask_in() to the ABI<br>
  ac: add load_resource() to the ABI<br>
  ac/nir: remove useless bitcast in load_tess_coord()<br>
  ac: remove unused parameters in abi::load_tess_coord()<br>
  ac/nir: drop nir_to_llvm_context from emit_f2f16()<br>
  ac/nir: drop nir_to_llvm_context from<br>
    visit_load_local_invocation_<wbr>index()<br>
  ac/nir: drop nir_to_llvm_context from cast_ptr()<br>
  ac/nir: drop nir_to_llvm_context from visit_load_push_constant()<br>
  ac/nir: drop nir_to_llvm_context from visit_vulkan_resource_reindex(<wbr>)<br>
  ac/nir: drop nir_to_llvm_context from visit_var_atomic()<br>
  ac/nir: drop nir_to_llvm_context from glsl_to_llvm_type()<br>
  ac/nir: remove nir_to_llvm_context::builder<br>
  ac/nir: remove nir_to_llvm_context::module<br>
  ac/nir: remove backlink to nir_to_llvm_context<br>
<br>
 src/amd/common/ac_nir_to_llvm.<wbr>c          | 407 +++++++++++++++---------------<wbr>-<br>
 src/amd/common/ac_shader_abi.h           |  23 +-<br>
 src/gallium/drivers/radeonsi/<wbr>si_shader.c |  12 +-<br>
 3 files changed, 224 insertions(+), 218 deletions(-)<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
2.16.1<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>
</font></span></blockquote></div><br></div>