Kenneth Graunke <kenn...@whitecape.org> writes:

> On 12/07/2012 02:08 PM, Eric Anholt wrote:
>> diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp 
>> b/src/mesa/drivers/dri/i965/brw_fs.cpp
>> index dbf48f8..b4e8d68 100644
>> --- a/src/mesa/drivers/dri/i965/brw_fs.cpp
>> +++ b/src/mesa/drivers/dri/i965/brw_fs.cpp
>> @@ -2434,6 +2434,14 @@ fs_visitor::run()
>>         else
>>       emit_interpolation_setup_gen6();
>>
>> +      /* On gen6+, we handle discards by keeping track of the still-live
>> +       * pixels in f0.1.  If so, initialize it with the dispatched pixels.
>
> The wording "If so" doesn't make sense to me.  Perhaps just "Initialize 
> it with the dispatched pixels."  The fact that we only do so for 
> programs that use discards is obvious from the code and makes a lot of 
> sense.

I had meant "if we are gen6, then do the setup".  Except that it applies
on non-gen6 too, since I made the change there as well!  Fixed.

> Also: does this new approach work in SIMD16 mode?  (Is the f0.1 wide 
> enough?)  It looks like you've removed the code to fail 16-wide compiles 
> that use discard.  I wasn't sure if that was intentional.

Yeah, that was intentional: we can use predication now instead of a
conditional.  This should fix any KIL performance regression pre-gen6,
though I haven't seen one reported.

Attachment: pgpCPctZzXPR1.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