On Fri, 17 Feb 2023 19:33:07 GMT, Joe Darcy wrote:
>> src/java.base/share/classes/java/lang/FdLibm.java line 458:
>>
>>> 456: case 0, 1 -> y; // atan(+/-0, +anything)
>>> = +/-0
>>> 457: case 2-> Math.PI + tiny; // atan(+0, -anything)
>>>
On Fri, 17 Feb 2023 14:18:25 GMT, Raffaello Giulietti
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 commi
On Fri, 17 Feb 2023 21:12:25 GMT, Joe Darcy wrote:
> Well spotted; corrected the code and added additional tests to catch that
> case. Thanks.
PS From some quick manual inspections, the other recent ports of FDLIBM methods
don't look vulnerable to this kind of problem. However, as follow-up wo
On Fri, 17 Feb 2023 13:50:17 GMT, Raffaello Giulietti
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 commi
On Fri, 17 Feb 2023 14:02:35 GMT, Raffaello Giulietti
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 commi
On Fri, 17 Feb 2023 13:48:32 GMT, Raffaello Giulietti
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 commi
On Fri, 17 Feb 2023 04:18:30 GMT, Joe Darcy wrote:
>> Working down the porting list, next stop, atan2.
>
> 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
> Working down the porting list, next stop, atan2.
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 re
