Matt Turner <matts...@gmail.com> writes: > On Wed, Apr 10, 2013 at 11:54 AM, Eric Anholt <e...@anholt.net> wrote: >> These checks were all over, and every time I wrote one I had to try to >> decide again what the cases were for partial updates. >> --- >> src/mesa/drivers/dri/i965/brw_fs.cpp | 35 >> +++++++++++--------- >> src/mesa/drivers/dri/i965/brw_fs.h | 1 + >> .../drivers/dri/i965/brw_fs_copy_propagation.cpp | 4 +-- >> src/mesa/drivers/dri/i965/brw_fs_cse.cpp | 3 +- >> .../drivers/dri/i965/brw_fs_live_variables.cpp | 4 +-- >> 5 files changed, 24 insertions(+), 23 deletions(-) >> >> diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp >> b/src/mesa/drivers/dri/i965/brw_fs.cpp >> index c12ba45..331e1ea 100644 >> --- a/src/mesa/drivers/dri/i965/brw_fs.cpp >> +++ b/src/mesa/drivers/dri/i965/brw_fs.cpp >> @@ -2662,10 +2670,7 @@ fs_visitor::get_instruction_generating_reg(fs_inst >> *start, >> fs_reg reg) >> { >> if (end == start || >> - end->predicate || >> - end->force_uncompressed || >> - end->force_sechalf || >> - reg.reladdr || > > Is the removal of reg.reladdr intentional?
Oops. Thanks for paying close attention.
pgpzcOrIJ2nZx.pgp
Description: PGP signature
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev