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

Vladimir Kempik vkempik at openjdk.org
Thu Nov 9 14:12:14 UTC 2023


On Thu, 9 Nov 2023 14:04:01 GMT, Hamlin Li <mli at openjdk.org> wrote:

>> 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.

@Hamlin-Li do you have GHA enabled for your repo ? It would be nice to let tier1 in gha pass to check updated test on all x86 platforms

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

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


More information about the hotspot-compiler-dev mailing list