On Fri, 3 Mar 2023 01:02:40 GMT, Vladimir Kozlov wrote:
> Hotspot changes are okay but I'm a bit confused about what the hotspot code
> will now be used for?
I'm not 100% positive if the current __kernel_rem_pio2 code would be in use.
IIRC, back when we used the fsin/fcos instructions to intri
On Fri, 3 Mar 2023 00:38:18 GMT, David Holmes wrote:
> Actually this is really my lack of understanding about the current code: why
> do we intrinsify `Math` but not `StrictMath`?
In brief, the Math methods are allowed implementation flexibility in terms of
their algorithm but the StrictMath m
On Fri, 3 Mar 2023 00:31:12 GMT, David Holmes wrote:
> Hotspot changes are okay but I'm a bit confused about what the hotspot code
> will now be used for?
`SharedRuntime::*` runtime math functions are used on platforms where there are
no HW instructions or intrinsics (Zero VM). JIT compiled co
On Thu, 2 Mar 2023 19:55:39 GMT, Joe Darcy wrote:
>> While the review of https://github.com/openjdk/jdk/pull/12800 finishes up, I
>> thought I'd get out for the review the next phase of the FDLIBM port:
>> removing the FDLIBM C sources from the repo.
>>
>> A repo with the changes for JDK-83020
On Thu, 2 Mar 2023 19:55:39 GMT, Joe Darcy wrote:
>> While the review of https://github.com/openjdk/jdk/pull/12800 finishes up, I
>> thought I'd get out for the review the next phase of the FDLIBM port:
>> removing the FDLIBM C sources from the repo.
>>
>> A repo with the changes for JDK-83020
On Thu, 2 Mar 2023 19:55:39 GMT, Joe Darcy wrote:
>> While the review of https://github.com/openjdk/jdk/pull/12800 finishes up, I
>> thought I'd get out for the review the next phase of the FDLIBM port:
>> removing the FDLIBM C sources from the repo.
>>
>> A repo with the changes for JDK-83020
> While the review of https://github.com/openjdk/jdk/pull/12800 finishes up, I
> thought I'd get out for the review the next phase of the FDLIBM port:
> removing the FDLIBM C sources from the repo.
>
> A repo with the changes for JDK-8302027 and this PR successful build on the
> default set of