<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Apr 7, 2016 at 2:30 PM, 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 Tue, Mar 22, 2016 at 3:33 PM, Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>> wrote:<br>
> The subnr field is in bytes so we don't need to multiply by type_sz.<br>
> ---<br>
>  src/mesa/drivers/dri/i965/brw_fs.cpp | 2 +-<br>
>  1 file changed, 1 insertion(+), 1 deletion(-)<br>
><br>
> diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp b/src/mesa/drivers/dri/i965/brw_fs.cpp<br>
> index 7d15794..d41c8a8 100644<br>
> --- a/src/mesa/drivers/dri/i965/brw_fs.cpp<br>
> +++ b/src/mesa/drivers/dri/i965/brw_fs.cpp<br>
> @@ -870,7 +870,7 @@ fs_inst::regs_read(int arg) const<br>
>               * unread portion at the beginning.<br>
>               */<br>
>              if (src[0].subnr)<br>
> -               region_length += src[0].subnr * type_sz(src[0].type);<br>
> +               region_length += src[0].subnr;<br>
<br>
</span>Looks correct... so is this code just unused today?<br>
</blockquote></div><br></div><div class="gmail_extra">I think so.  The uses of MOV_INDIRECT are pretty simple today.<br></div></div>