RFR: 8346931: Replace divisions by zero in sharedRuntimeTrans.cpp [v2]

[Kim Barrett]

On Mon, 24 Mar 2025 13:04:46 GMT, Matthias Baesken <mbaesken at openjdk.org> wrote:

>> src/hotspot/share/runtime/sharedRuntimeTrans.cpp line 518:
>> 
>>> 516:       z = ax;                   /*x is +-0,+-inf,+-1*/
>>> 517:       if(hy<0) {
>>> 518:         if (z == 0.0) {
>> 
>> Maybe s/z == 0.0/ix == 0/ ?
>
> Hi Kim, why ix and not x ? Is it a typo?

Not a typo, I really meant `ix`.  The point is that testing the `int` typed `ix` for zero is no worse, and might
be better than, testing a `double` typed value for zero.  And I suggested `ix` rather than `hx` because `ix`
has the sign bit stripped off.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/24136#discussion_r2010883179