RFR: 8319705: RISC-V: signumF/D intrinsics fails compiler/intrinsics/math/TestSignumIntrinsic.java [v3]

[Hamlin Li]

On Thu, 9 Nov 2023 13:10:31 GMT, Ilya Gavrilin <igavrilin at openjdk.org> wrote:

>> Hamlin Li has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains five additional commits since the last revision:
>> 
>>  - use t0 directly instead of get another tmp from reg allocator
>>  - Merge branch 'master' into fix-signum
>>  - remove rFlagsReg
>>  - Kill cr
>>  - Initial commit
>
> Hi, could you, please,  add this small update to jtreg: hotspot/jtreg/compiler/intrinsics/math/TestSignumIntrinsic.java:
> 
> diff --git a/test/hotspot/jtreg/compiler/intrinsics/math/TestSignumIntrinsic.java b/test/hotspot/jtreg/compiler/intrinsics/math/TestSignumIntrinsic.java
> index 65d261bb8e2..a5546ba9e5c 100644
> --- a/test/hotspot/jtreg/compiler/intrinsics/math/TestSignumIntrinsic.java
> +++ b/test/hotspot/jtreg/compiler/intrinsics/math/TestSignumIntrinsic.java
> @@ -35,6 +35,10 @@
>   *      -XX:+IgnoreUnrecognizedVMOptions -XX:+UnlockDiagnosticVMOptions
>   *      -XX:-UseSignumIntrinsic -XX:+UseCopySignIntrinsic
>   *      compiler.intrinsics.math.TestSignumIntrinsic
> + * @run main/othervm
> + *      -XX:+IgnoreUnrecognizedVMOptions -XX:+UnlockDiagnosticVMOptions
> + *      -Xcomp -XX:+UseSignumIntrinsic
> + *      compiler.intrinsics.math.TestSignumIntrinsic
>   */
>  
>  package compiler.intrinsics.math;

Thanks @Ilyagavrilin @VladimirKempik, test modified.

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

PR Comment: https://git.openjdk.org/jdk/pull/16557#issuecomment-1803893467