RFR: JDK-8302028: Port fdlibm atan2 to Java [v2]

Joe Darcy darcy at openjdk.org
Fri Feb 17 21:17:13 UTC 2023


On Fri, 17 Feb 2023 13:50:17 GMT, Raffaello Giulietti <rgiulietti at openjdk.org> wrote:

>> Joe Darcy has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains three additional commits since the last revision:
>> 
>>  - Update regression test.
>>  - Merge branch 'master' into JDK-8302028
>>  - JDK-8302028: Port fdlibm atan2 to Java
>
> src/java.base/share/classes/java/lang/FdLibm.java line 447:
> 
>> 445:             ly = __LO(y);
>> 446:             if (((ix | ((lx | -lx) >> 31)) > 0x7ff0_0000)||
>> 447:                 ((iy |((ly | - ly) >> 31)) > 0x7ff0_0000))    // x or y is NaN
> 
> I think that `>> 31` must be replaced by `>>> 31`.

Well spotted; corrected the code and added additional tests to catch that case. Thanks.

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

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


More information about the core-libs-dev mailing list