<p dir="ltr"><br>
On Aug 27, 2015 10:33 PM, "Kenneth Graunke" <<a href="mailto:kenneth@whitecape.org">kenneth@whitecape.org</a>> wrote:<br>
><br>
> On Thursday, August 27, 2015 10:17:24 PM Matt Turner wrote:<br>
> > Noticed when debugging things that lead to the next patch.<br>
> ><br>
> > On G45 (and presumably ILK) this helps register coalescing:<br>
> ><br>
> > total instructions in shared programs: 4077373 -> 4077340 (-0.00%)<br>
> > instructions in affected programs:     43751 -> 43718 (-0.08%)<br>
> > helped:                                52<br>
> > HURT:                                  2<br>
> > ---<br>
> >  src/mesa/drivers/dri/i965/brw_fs.cpp   | 4 +++-<br>
> >  src/mesa/drivers/dri/i965/brw_vec4.cpp | 5 +++--<br>
> >  2 files changed, 6 insertions(+), 3 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 81009a0..269914d 100644<br>
> > --- a/src/mesa/drivers/dri/i965/brw_fs.cpp<br>
> > +++ b/src/mesa/drivers/dri/i965/brw_fs.cpp<br>
> > @@ -427,7 +427,9 @@ fs_reg::equals(const fs_reg &r) const<br>
> >             negate == r.negate &&<br>
> >             abs == r.abs &&<br>
> >             !reladdr && !r.reladdr &&<br>
> > -           memcmp(&fixed_hw_reg, &r.fixed_hw_reg, sizeof(fixed_hw_reg)) == 0 &&<br>
> > +           ((file != HW_REG && file != IMM) ||<br>
> > +            memcmp(&fixed_hw_reg, &r.fixed_hw_reg,<br>
> > +                   sizeof(fixed_hw_reg)) == 0) &&<br>
> >             stride == r.stride);<br>
> >  }<br>
> ><br>
> > diff --git a/src/mesa/drivers/dri/i965/brw_vec4.cpp b/src/mesa/drivers/dri/i965/brw_vec4.cpp<br>
> > index f18915a..b97b6c1 100644<br>
> > --- a/src/mesa/drivers/dri/i965/brw_vec4.cpp<br>
> > +++ b/src/mesa/drivers/dri/i965/brw_vec4.cpp<br>
> > @@ -216,8 +216,9 @@ dst_reg::equals(const dst_reg &r) const<br>
> >             writemask == r.writemask &&<br>
> >             (reladdr == r.reladdr ||<br>
> >              (reladdr && r.reladdr && reladdr->equals(*r.reladdr))) &&<br>
> > -           memcmp(&fixed_hw_reg, &r.fixed_hw_reg,<br>
> > -                  sizeof(fixed_hw_reg)) == 0);<br>
> > +           ((file != HW_REG && file != IMM) ||<br>
> > +            memcmp(&fixed_hw_reg, &r.fixed_hw_reg,<br>
> > +                   sizeof(fixed_hw_reg)) == 0));<br>
> >  }<br>
> ><br>
> >  bool<br>
> ><br>
><br>
> Nice!  The updated live range handling makes a lot more sense - I always<br>
> thought the old stuff was bunk...nice to see it was just papering over bugs.<br>
><br>
> Series is:<br>
> Reviewed-by: Kenneth Graunke <<a href="mailto:kenneth@whitecape.org">kenneth@whitecape.org</a>></p>
<p dir="ltr">Bah! Beat me to it...</p>
<p dir="ltr">Reviewed-by: Jason Ekstrand <<a href="mailto:jason.ekstrand@intel.com">jason.ekstrand@intel.com</a>></p>
<p dir="ltr">> Also, thank you for producing a series that, despite touching a complex<br>
> piece of code, I was able to review completely in 15 minutes!<br>
><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>
><br>
</p>