Paul Berry <stereotype...@gmail.com> writes:

> On 31 October 2013 18:57, Matt Turner <matts...@gmail.com> wrote:
>
>> Before this patch, the following code would not be optimized even though
>> the first two instructions were common to the then and else blocks:
>>
>>    (+f0) IF
>>    MOV dst0 ...
>>    MOV dst1 ...
>>    MOV dst2 ...
>>    ELSE
>>    MOV dst0 ...
>>    MOV dst1 ...
>>    MOV dst3 ...
>>    ENDIF
>>
>> This commit extends the peephole to handle this case.
>>
>> No shader-db changes.
>> ---
>> This patch made a much bigger difference when pulling instructions from the
>> ends of the "then" and "else" blocks, because often we were able to compute
>> to MRF.
>>
>
> This patch is:
>
> Reviewed-by: Paul Berry <stereotype...@gmail.com>
>
> I'd be just as happy if you wanted to squash patches 3 and 4 together into
> a single patch.

I was going to say the same thing for my r-b.

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