On Wed, 1 Mar 2023 05:28:34 GMT, Joe Darcy <da...@openjdk.org> wrote:

> Last and certainly not least in the port of FDLIBM to Java, the 
> transcendental methods for sin, cos, and tan.
> 
> Some more tests are to be written in the StrictMath directory to verify that 
> the StrictMath algorihtm for sin/cos/tan is being used rather than a 
> different one. However, I wanted to get the rest of the change out for review 
> first.
> 
> The sin/cos/tan methods are grouped together since they share the same 
> argument reduction logic. Argument reduction is the process of mapping an 
> argument of a function to an argument in a restricted range (and possibly 
> returning some function of the reduced argument). For sin, cos, and tan, 
> since they are fundamentally periodic with respect to a multiple of pi, 
> argument reduction is done to find the remainder of the original argument 
> with respect to pi/2.

This pull request has now been integrated.

Changeset: 1bb39a95
Author:    Joe Darcy <da...@openjdk.org>
URL:       
https://git.openjdk.org/jdk/commit/1bb39a95eb42c7f68ba59ac2809717cd7b1df7bc
Stats:     2305 lines in 7 files changed: 2283 ins; 12 del; 10 mod

8302027: Port fdlibm trig functions (sin, cos, tan) to Java

Reviewed-by: bpb

-------------

PR: https://git.openjdk.org/jdk/pull/12800

Reply via email to