diff options
author | Ilia Mirkin <[email protected]> | 2016-03-12 21:26:21 -0500 |
---|---|---|
committer | Ilia Mirkin <[email protected]> | 2016-03-13 13:17:24 -0400 |
commit | c1e4a6bfbf015801c6a8b0ae694482421a22c2d9 (patch) | |
tree | 207eccde3645c53393c2d174f24fb3e381ae5fab /src/gallium/drivers/r300/r300_resource.c | |
parent | b3e7fb52349848b24f005c07859bc43691bd64bd (diff) |
nv50,nvc0: handle SQRT lowering inside the driver
First off, st/mesa lowers DSQRT incorrectly (it uses CMP to attempt to
find out whether the input is less than 0). Secondly the current
approach (x * rsq(x)) behaves poorly for x = inf - a NaN is produced
instead of inf.
Instead we switch to the less accurate rcp(rsq(x)) method - this behaves
nicely for all valid inputs. We still don't do this for DSQRT since the
RSQ/RCP ops are *really* inaccurate, and don't even have Newton-Raphson
steps right now. Eventually we should have a separate library function
for DSQRT that does it more precisely (and perhaps move this lowering to
the post-opt phase).
This fixes a number of dEQP precision tests that were expecting better
behavior for infinite inputs.
Signed-off-by: Ilia Mirkin <[email protected]>
Reviewed-by: Samuel Pitoiset <[email protected]>
Tested-by: Samuel Pitoiset <[email protected]>
Diffstat (limited to 'src/gallium/drivers/r300/r300_resource.c')
0 files changed, 0 insertions, 0 deletions