RFR: 8261671: X86 I2L conversion can be skipped for certain masked positive values [v3]

[Vladimir Kozlov]

On Thu, 4 Mar 2021 21:15:27 GMT, Marcus G K Williams <github.com+168222+mgkwill at openjdk.org> wrote:

>> compiler/intrinsics/bmi/TestBzhiI2L.java test failed:
>>  stdout: [For random generator using seed: -6986231732124296613
>> To re-run test with same seed value please add "-Djdk.test.lib.random.seed=-6986231732124296613" to command line.
>> ];
>>  stderr: [Exception in thread "main" java.lang.UnsupportedOperationException
>> 	at compiler.intrinsics.bmi.Expr.longExpr(Expr.java:97)
>> 	at compiler.intrinsics.bmi.BMITestRunner$Executor.runUnaryLongRegTest(BMITestRunner.java:280)
>> 	at compiler.intrinsics.bmi.BMITestRunner$Executor.runTests(BMITestRunner.java:219)
>> 	at compiler.intrinsics.bmi.BMITestRunner$Executor.main(BMITestRunner.java:190)
>> ]
>>  exitValue = 1
>> 
>> java.lang.RuntimeException: Expected to get exit value of [0]
>> 
>> 	at jdk.test.lib.process.OutputAnalyzer.shouldHaveExitValue(OutputAnalyzer.java:474)
>> 	at compiler.intrinsics.bmi.BMITestRunner.runTest(BMITestRunner.java:151)
>> 	at compiler.intrinsics.bmi.BMITestRunner.runTests(BMITestRunner.java:87)
>> 	at compiler.intrinsics.bmi.TestBzhiI2L.main(TestBzhiI2L.java:49)
>
>> compiler/intrinsics/bmi/TestBzhiI2L.java test failed:
>> 
>> ```
>>  stdout: [For random generator using seed: -6986231732124296613
>> To re-run test with same seed value please add "-Djdk.test.lib.random.seed=-6986231732124296613" to command line.
>> ];
>>  stderr: [Exception in thread "main" java.lang.UnsupportedOperationException
>> 	at compiler.intrinsics.bmi.Expr.longExpr(Expr.java:97)
>> 	at compiler.intrinsics.bmi.BMITestRunner$Executor.runUnaryLongRegTest(BMITestRunner.java:280)
>> 	at compiler.intrinsics.bmi.BMITestRunner$Executor.runTests(BMITestRunner.java:219)
>> 	at compiler.intrinsics.bmi.BMITestRunner$Executor.main(BMITestRunner.java:190)
>> ]
>>  exitValue = 1
>> 
>> java.lang.RuntimeException: Expected to get exit value of [0]
>> 
>> 	at jdk.test.lib.process.OutputAnalyzer.shouldHaveExitValue(OutputAnalyzer.java:474)
>> 	at compiler.intrinsics.bmi.BMITestRunner.runTest(BMITestRunner.java:151)
>> 	at compiler.intrinsics.bmi.BMITestRunner.runTests(BMITestRunner.java:87)
>> 	at compiler.intrinsics.bmi.TestBzhiI2L.main(TestBzhiI2L.java:49)
>> ```
> 
> Thanks @vnkozlov. Interesting. I'll take a look using "-Djdk.test.lib.random.seed=-6986231732124296613"

Note, the test only failed on machines which do not support BMI2: AMD EPYC and Aarch64.

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

PR: https://git.openjdk.java.net/jdk/pull/2590