<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Sep 5, 2014 at 5:25 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 Fri, Sep 5, 2014 at 5:09 PM, Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>> wrote:<br>
> Uniform values are in the UNIFORM register file, not the GRF register file.<br>
> Looking in virtual_grf_sizes makes no sense and only makes the output of<br>
> dump_instructions confusint.<br>
<br>
</span>typo<span class=""><br></span></blockquote><div> <br></div><div>Yup, typing too fast.<br></div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
><br>
> Signed-off-by: Jason Ekstrand <<a href="mailto:jason.ekstrand@intel.com">jason.ekstrand@intel.com</a>><br>
> ---<br>
>  src/mesa/drivers/dri/i965/brw_fs.cpp | 3 +--<br>
>  1 file changed, 1 insertion(+), 2 deletions(-)<br>
><br>
> diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp b/src/mesa/drivers/dri/i965/brw_fs.cpp<br>
> index 4665dca..353f8c0 100644<br>
> --- a/src/mesa/drivers/dri/i965/brw_fs.cpp<br>
> +++ b/src/mesa/drivers/dri/i965/brw_fs.cpp<br>
> @@ -2968,8 +2968,7 @@ fs_visitor::dump_instruction(backend_instruction *be_inst, FILE *file)<br>
>           fprintf(file, "u%d", inst->src[i].reg + inst->src[i].reg_offset);<br>
>           if (inst->src[i].reladdr) {<br>
>              fprintf(file, "+reladdr");<br>
> -         } else if (virtual_grf_sizes[inst->src[i].reg] != 1 ||<br>
> -             inst->src[i].subreg_offset) {<br>
> +         } else if (inst->src[i].subreg_offset) {<br>
<br>
</span>Can uniforms can have subreg offsets at all?<br>
<br>
In any case, the old code is wrong.<br>
<br>
Reviewed-by: Matt Turner <<a href="mailto:mattst88@gmail.com">mattst88@gmail.com</a>><br>
</blockquote></div><br></div></div>