Integrated: 8319705: RISC-V: signumF/D intrinsics fails compiler/intrinsics/math/TestSignumIntrinsic.java

[Hamlin Li]

On Wed, 8 Nov 2023 12:05:17 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

This pull request has now been integrated.

Changeset: 7b971c1f
Author:    Hamlin Li <mli at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/7b971c1fe82c7570621951ebe0ca9120bedbd27d
Stats:     28 lines in 4 files changed: 4 ins; 8 del; 16 mod

8319705: RISC-V: signumF/D intrinsics fails compiler/intrinsics/math/TestSignumIntrinsic.java

Reviewed-by: fyang, vkempik

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

PR: https://git.openjdk.org/jdk/pull/16557