<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, May 31, 2018 at 10:31 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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Thu, May 24, 2018 at 2:56 PM, Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>> wrote:<br>
> From: Francisco Jerez <<a href="mailto:currojerez@riseup.net">currojerez@riseup.net</a>><br>
><br>
> Reviewed-by: Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>><br>
> ---<br>
>  src/intel/compiler/brw_fs.cpp | 8 +++++++-<br>
>  1 file changed, 7 insertions(+), 1 deletion(-)<br>
><br>
> diff --git a/src/intel/compiler/brw_fs.<wbr>cpp b/src/intel/compiler/brw_fs.<wbr>cpp<br>
> index f583900..ae59716 100644<br>
> --- a/src/intel/compiler/brw_fs.<wbr>cpp<br>
> +++ b/src/intel/compiler/brw_fs.<wbr>cpp<br>
> @@ -5088,8 +5088,14 @@ get_fpu_lowered_simd_width(<wbr>const struct gen_device_info *devinfo,<br>
>              type_sz(inst->dst.type) == 4 && inst->dst.stride == 1 &&<br>
>              type_sz(inst->src[i].type) == 2 && inst->src[i].stride == 1;<br>
><br>
> +         /* We check size_read(i) against size_written instead of REG_SIZE<br>
> +          * because we want to properly handle SIMD32.  In SIMD32, you can end<br>
> +          * up which writes 4 registers and a source that reads 2 registers<br>
<br>
</span>This reads a little strangely. Maybe "you can end up with writes to 4<br>
<div class="HOEnZb"><div class="h5">registers and a source that reads 2 registers"<br></div></div></blockquote><div><br></div><div>Yeah, I think that's better.  I wouldn't say it's now "good" but it is better.<br></div></div><br></div></div>