[Mesa-dev] [PATCH v2 03/15] i965/fs: Fix regs_read() for MOV_INDIRECT with a non-zero subnr

[Matt Turner]

On Tue, Mar 22, 2016 at 3:33 PM, Jason Ekstrand <jason at jlekstrand.net> wrote:
> The subnr field is in bytes so we don't need to multiply by type_sz.
> ---
>  src/mesa/drivers/dri/i965/brw_fs.cpp | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp b/src/mesa/drivers/dri/i965/brw_fs.cpp
> index 7d15794..d41c8a8 100644
> --- a/src/mesa/drivers/dri/i965/brw_fs.cpp
> +++ b/src/mesa/drivers/dri/i965/brw_fs.cpp
> @@ -870,7 +870,7 @@ fs_inst::regs_read(int arg) const
>               * unread portion at the beginning.
>               */
>              if (src[0].subnr)
> -               region_length += src[0].subnr * type_sz(src[0].type);
> +               region_length += src[0].subnr;

Looks correct... so is this code just unused today?