<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, May 31, 2018 at 11:28 AM, Jason Ekstrand <span dir="ltr"><<a href="mailto:jason@jlekstrand.net" target="_blank">jason@jlekstrand.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div class="h5"><div class="gmail_extra"><div class="gmail_quote">On Thu, May 31, 2018 at 10:43 AM, Matt Turner <span dir="ltr"><<a href="mailto:mattst88@gmail.com" target="_blank">mattst88@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div class="m_5388991046768319842gmail-HOEnZb"><div class="m_5388991046768319842gmail-h5">On Thu, May 24, 2018 at 2:56 PM, Jason Ekstrand <<a href="mailto:jason@jlekstrand.net" target="_blank">jason@jlekstrand.net</a>> wrote:<br>
> ---<br>
>  src/intel/compiler/brw_fs.cpp | 25 ++++++-------------------<br>
>  1 file changed, 6 insertions(+), 19 deletions(-)<br>
><br>
> diff --git a/src/intel/compiler/brw_fs.cp<wbr>p b/src/intel/compiler/brw_fs.cp<wbr>p<br>
> index 1f87f06..7e532af 100644<br>
> --- a/src/intel/compiler/brw_fs.cp<wbr>p<br>
> +++ b/src/intel/compiler/brw_fs.cp<wbr>p<br>
> @@ -3220,21 +3220,14 @@ fs_visitor::emit_repclear_shad<wbr>er()<br>
>     brw_wm_prog_key *key = (brw_wm_prog_key*) this->key;<br>
>     int base_mrf = 0;<br>
>     int color_mrf = base_mrf + 2;<br>
> -   fs_inst *mov;<br>
><br>
> -   if (uniforms > 0) {<br>
> -      mov = bld.exec_all().group(4, 0)<br>
> -               .MOV(brw_message_reg(color_mr<wbr>f),<br>
> -                    fs_reg(UNIFORM, 0, BRW_REGISTER_TYPE_F));<br>
> -   } else {<br>
> -      struct brw_reg reg =<br>
> -         brw_reg(BRW_GENERAL_REGISTER_<wbr>FILE, 2, 3, 0, 0, BRW_REGISTER_TYPE_F,<br>
> -                 BRW_VERTICAL_STRIDE_8, BRW_WIDTH_2, BRW_HORIZONTAL_STRIDE_4,<br>
> -                 BRW_SWIZZLE_XYZW, WRITEMASK_XYZW);<br>
> +   const struct brw_reg color_in_reg =<br>
> +      brw_reg(BRW_GENERAL_REGISTER_F<wbr>ILE, 2, 3, 0, 0, BRW_REGISTER_TYPE_F,<br>
> +              BRW_VERTICAL_STRIDE_8, BRW_WIDTH_2, BRW_HORIZONTAL_STRIDE_4,<br>
> +              BRW_SWIZZLE_XYZW, WRITEMASK_XYZW);<br>
><br>
> -      mov = bld.exec_all().group(4, 0)<br>
> -               .MOV(vec4(brw_message_reg(col<wbr>or_mrf)), fs_reg(reg));<br>
> -   }<br>
> +   bld.exec_all().group(4, 0)<br>
> +      .MOV(vec4(brw_message_reg(colo<wbr>r_mrf)), fs_reg(color_in_reg));<br>
><br>
>     fs_inst *write = NULL;<br>
>     if (key->nr_color_regions == 1) {<br>
> @@ -3273,12 +3266,6 @@ fs_visitor::emit_repclear_shad<wbr>er()<br>
><br>
>     assign_constant_locations();<br>
>     assign_curb_setup();<br>
> -<br>
> -   /* Now that we have the uniform assigned, go ahead and force it to a vec4. */<br>
> -   if (uniforms > 0) {<br>
> -      assert(mov->src[0].file == FIXED_GRF);<br>
> -      mov->src[0] = brw_vec4_grf(mov->src[0].nr, 0);<br>
> -   }<br>
>  }<br>
><br>
>  /**<br>
> --<br>
<br>
</div></div>Some explanation required: why are we making this change, how is it<br>
safe (i.e., did something earlier in the series make this possible?).<br>
</blockquote></div></div><div class="gmail_extra"><br></div></div></div><div class="gmail_extra">It hasn't been used since</div><div class="gmail_extra"><br></div><div class="gmail_extra">commit 46e1132b80f01f4670dac4e393d36b<wbr>0e9266b02f<br>Author: Topi Pohjolainen <<a href="mailto:topi.pohjolainen@intel.com" target="_blank">topi.pohjolainen@intel.com</a>><br>Date:   Sun May 15 07:43:39 2016 +0300<br><br>    i965/blorp: Use flat inputs instead of uniforms<br>    <br>    v2 (Jason): Use LOAD_INPUT() macro<br>    <br>    Signed-off-by: Topi Pohjolainen <<a href="mailto:topi.pohjolainen@intel.com" target="_blank">topi.pohjolainen@intel.com</a>><br>    Reviewed-by: Jason Ekstrand <<a href="mailto:jason@jlekstrand.net" target="_blank">jason@jlekstrand.net</a>><br><br></div></div>
</blockquote></div></div><div class="gmail_extra"><br></div><div class="gmail_extra">That said, SIMD32 with repclear has shown itself to be a bit more annoying than first thought so I think it's probably best to drop 51 and 52.  They were added somewhat last-minute.<br></div><div class="gmail_extra"><br></div><div class="gmail_extra">--Jason<br></div></div>