<p dir="ltr"><br>
On Jun 29, 2015 11:21 AM, "Francisco Jerez" <<a href="mailto:currojerez@riseup.net">currojerez@riseup.net</a>> wrote:<br>
><br>
> Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>> writes:<br>
><br>
> > Reviewed-by: Iago Toral Quiroga <<a href="mailto:itoral@igalia.com">itoral@igalia.com</a>><br>
> > Reviewed-by: Topi Pohjolainen <<a href="mailto:topi.pohjolainen@intel.com">topi.pohjolainen@intel.com</a>><br>
> > ---<br>
> >  src/mesa/drivers/dri/i965/brw_fs.cpp | 6 ++++++<br>
> >  1 file changed, 6 insertions(+)<br>
> ><br>
> > diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp b/src/mesa/drivers/dri/i965/brw_fs.cpp<br>
> > index 589b74c..6cf9e96 100644<br>
> > --- a/src/mesa/drivers/dri/i965/brw_fs.cpp<br>
> > +++ b/src/mesa/drivers/dri/i965/brw_fs.cpp<br>
> > @@ -726,6 +726,12 @@ fs_inst::regs_read(int arg) const<br>
> >           return exec_size / 4;<br>
> >        break;<br>
> ><br>
> > +   case FS_OPCODE_PIXEL_X:<br>
> > +   case FS_OPCODE_PIXEL_Y:<br>
> > +      if (arg == 0)<br>
> > +         return 2;<br>
> > +      break;<br>
> > +<br>
><br>
> This doesn't look right.  AFAICT PIXEL_X/Y take two exec_size-wide<br>
> components of UW type (interleaved for each subspan, but that doesn't<br>
> matter here), i.e. two registers in SIMD16 mode but only one register in<br>
> SIMD8 mode.</p>
<p dir="ltr">Yeah, I think you're right. Not sure why piglit didn't catch that. I'll get it fixed.<br>
--Jason</p>
<p dir="ltr">> >     default:<br>
> >        if (is_tex() && arg == 0 && src[0].file == GRF)<br>
> >           return mlen;<br>
> > --<br>
> > 2.4.3<br>
> ><br>
> > _______________________________________________<br>
> > mesa-dev mailing list<br>
> > <a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a><br>
> > <a href="http://lists.freedesktop.org/mailman/listinfo/mesa-dev">http://lists.freedesktop.org/mailman/listinfo/mesa-dev</a><br>
</p>