RFR: 8319705: RISC-V: signumF/D intrinsics fails compiler/intrinsics/math/TestSignumIntrinsic.java [v4]
Vladimir Kempik
vkempik at openjdk.org
Thu Nov 9 14:24:07 UTC 2023
On Thu, 9 Nov 2023 14:06:54 GMT, Hamlin Li <mli at openjdk.org> wrote:
>> 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
Marked as reviewed by vkempik (Committer).
-------------
PR Review: https://git.openjdk.org/jdk/pull/16557#pullrequestreview-1722660970
More information about the hotspot-compiler-dev
mailing list