tree c29a058709eab2a5cf9a9472978e0e7859438cc1
parent f7c42b049e6bf0e0e0ba1489a3866cbdf3ab88e2
author Nicolas Capens <capn@google.com> 1644041954 -0500
committer Nicolas Capens <nicolascapens@google.com> 1644262216 +0000

Eliminate rr::Precision

Reactor now only has full-precision implementations of transcendental
functions, so there's no need for a precision argument.

The only exceptions are the reciprocal function, and reciprocal square
root. Those are very useful to keep as part of the Reactor API because
there are fast approximate instructions for these operations. For now,
the precision is selected using a boolean parameter. This is intended to
later be replaced by having just an approximate intrinsic, and a query
Caps::rcpIsFast() which will allow users of Reactor to determine whether
to use 1/x or Rcp(). Specifically, when Rcp() is actually implemented as
1/x it would be wasteful to apply the Newton–Raphson method to obtain a
more precise result (cf. Caps::fmaIsFast() which addresses a similar
problem).

Bug: b/169755552
Change-Id: I553746aeee3ac01f6c47e06da17de1f6bfcd7df6
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/62289
Kokoro-Result: kokoro <noreply+kokoro@google.com>
Reviewed-by: Alexis Hétu <sugoi@google.com>
Tested-by: Nicolas Capens <nicolascapens@google.com>