[Mesa-dev] [PATCH 7/9] nir: Fix broken fsat recognizer.

[Eric Anholt]

Connor Abbott <cwabbott0 at gmail.com> writes:

> On Sun, Feb 1, 2015 at 4:17 PM, Eric Anholt <eric at anholt.net> wrote:
>> We've probably never seen this ridiculous pattern in the wild, so it
>> didn't matter.
>
> I don't think this pattern is 'ridiculous' at all -- GLSL doesn't have
> a sat operation, so shaders wanting to do a saturate have to do this
> already. Rather, what probably saves us is that we already recognize
> this pattern in GLSL IR. Other than that,
>
> Reviewed-by: Connor Abbott <cwabbott0 at gmail.com>

I meant "we hadn't seen that this was a bug because nobody would clamp
the wrong way around like this."
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 818 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20150201/515d6989/attachment.sig>