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.

Attachment: pgpzcOrIJ2nZx.pgp
Description: PGP signature

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to