[jdk16] RFR: 8259227: C2 crashes with SIGFPE due to a division that floats above its zero check [v2]

[Christian Hagedorn]

On Thu, 7 Jan 2021 10:57:53 GMT, Tobias Hartmann <thartmann at openjdk.org> wrote:

>> Christian Hagedorn has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   
>>   
>>   Add additional runs without fixed seed
>
> test/hotspot/jtreg/compiler/loopopts/TestDivZeroDominatedBy.java line 31:
> 
>> 29:  * @requires vm.compiler2.enabled
>> 30:  * @run main/othervm -Xcomp -XX:-TieredCompilation -XX:CompileOnly=compiler/loopopts/TestDivZeroDominatedBy::test
>> 31:  *                   -XX:+UnlockDiagnosticVMOptions -XX:+StressGCM -XX:StressSeed=917280111 compiler.loopopts.TestDivZeroDominatedBy
> 
> Just wondering if the `StressSeed` should really be hardcoded? If things change in the VM, this seed might not longer reproduce the issue. Maybe add another `@run `with no fixed seed?

Good idea. I also updated the test from JDK-8257822 with such an additional run.

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

PR: https://git.openjdk.java.net/jdk16/pull/89