RFR: 6506405: Math.abs(float) is slow [v2]
Uwe Schindler
uschindler at apache.org
Thu Jul 8 07:09:01 UTC 2021
On a quick review of version 2, I noticed that the double constant is declared as integer with same value like the float one.
I think that's wrong, but I wonder why the asserts still pass.
Uwe
Am July 8, 2021 1:05:16 AM UTC schrieb Brian Burkhalter <bpb at openjdk.java.net>:
>> Please consider this change to make the `float` and `double` versions
>of `java.lang.Math.abs()` branch-free.
>
>Brian Burkhalter has updated the pull request incrementally with one
>additional commit since the last revision:
>
> 6506405: Add comments, use new consts for masking
>
>-------------
>
>Changes:
> - all: https://git.openjdk.java.net/jdk/pull/4711/files
>- new:
>https://git.openjdk.java.net/jdk/pull/4711/files/60d461fc..ba950f60
>
>Webrevs:
> - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=4711&range=01
> - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=4711&range=00-01
>
> Stats: 22 lines in 3 files changed: 17 ins; 0 del; 5 mod
> Patch: https://git.openjdk.java.net/jdk/pull/4711.diff
>Fetch: git fetch https://git.openjdk.java.net/jdk
>pull/4711/head:pull/4711
>
>PR: https://git.openjdk.java.net/jdk/pull/4711
More information about the core-libs-dev
mailing list