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

Eric Anholt eric at anholt.net
Sun Feb 1 15:54:54 PST 2015

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>

More information about the mesa-dev mailing list