r600: use min_dx10/max_dx10 instead of min/max
authorRoland Scheidegger <sroland@vmware.com>
Thu, 9 Nov 2017 18:37:54 +0000 (19:37 +0100)
committerRoland Scheidegger <sroland@vmware.com>
Wed, 15 Nov 2017 02:13:46 +0000 (03:13 +0100)
commitaab0bfc648bf1be50b81a25224970015f1dc78b8
tree4807515fd5818b2a0fdf3e1e2e8761e75cc86b7b
parent3ceee04a4f963694012ca7f9acd3c747a036472e
r600: use min_dx10/max_dx10 instead of min/max

I believe this is the safe thing to do, especially ever since the driver
actually generates NaNs for muls too.
The ISA docs are not very helpful here, however the dx10 versions will pick
a non-nan result over a NaN one (this is also the ieee754 behavior), whereas
the non-dx10 ones will pick the NaN (verified by newly changed piglit
isinf-and-isnan test).
Other "modern" drivers will most likely do the same.
This was shown to make some difference for bug 103544, albeit it is not
required to fix it.

Reviewed-by: Dave Airlie <airlied@redhat.com>
src/gallium/drivers/r600/r600_shader.c
src/gallium/drivers/r600/sb/sb_expr.cpp