RFR: 8319705: RISC-V: signumF/D intrinsics fails compiler/intrinsics/math/TestSignumIntrinsic.java [v4]
Hamlin Li
mli at openjdk.org
Thu Nov 9 14:06:54 UTC 2023
> Hi,
> Can you review the patch to fix the bug introduced by [JDK-8317971](https://bugs.openjdk.org/browse/JDK-8317971) which implemented signumF/D intrinsics.
> But seems the intrinsics failed since day 1 (I checkouted the commit to verify it.)
> Thanks
>
> ## Test
> hotspot tests:
>
> test/hotspot/jtreg/compiler/intrinsics/math/TestSignumIntrinsic.java
> test/hotspot/jtreg/compiler/c2/PolynomialRoot.java
>
>
> other jdk tests contains Math.signum(...):
>
> test/jdk/java/math/BigInteger/BigIntegerTest.java
> test/jdk/java/math/BigInteger/largeMemory/DivisionOverflow.java
> test/jdk/java/math/BigInteger/SerializationTests.java
> test/jdk/java/math/BigInteger/ModInvTime.java
> test/jdk/java/math/BigDecimal/SquareRootTests.java
> test/jdk/java/math/BigDecimal/DivideTests.java
> test/jdk/java/lang/Math/IeeeRecommendedTests.java
> test/jdk/java/lang/Math/IeeeRemainderTests.java
Hamlin Li has updated the pull request incrementally with one additional commit since the last revision:
test with -Xcomp
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/16557/files
- new: https://git.openjdk.org/jdk/pull/16557/files/85de9252..d32cbb9c
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=16557&range=03
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=16557&range=02-03
Stats: 4 lines in 1 file changed: 4 ins; 0 del; 0 mod
Patch: https://git.openjdk.org/jdk/pull/16557.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/16557/head:pull/16557
PR: https://git.openjdk.org/jdk/pull/16557
More information about the hotspot-compiler-dev
mailing list