RFR: 8335393: C2: assert(!had_error) failed: bad dominance [v2]

[Roland Westrelin]

On Wed, 24 Jul 2024 12:08:18 GMT, Christian Hagedorn <chagedorn at openjdk.org> wrote:

>> Roland Westrelin has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Update src/hotspot/share/opto/loopTransform.cpp
>>   
>>   Co-authored-by: Tobias Hartmann <tobias.hartmann at oracle.com>
>
> test/hotspot/jtreg/compiler/rangechecks/TestEmptyLoopDeadCast.java line 29:
> 
>> 27:  * @summary C2: assert(!had_error) failed: bad dominance
>> 28:  * @run main/othervm -XX:-TieredCompilation -XX:-UseOnStackReplacement -XX:-BackgroundCompilation -XX:-UseLoopPredicate
>> 29:  *                   -XX:LoopMaxUnroll=0 TestEmptyLoopDeadCast
> 
> You should either add `-XX:+IgnoreUnrecognizedVMOptions` or `@requires vm.compiler2.enabled` since you use C2-specific flags.

Right, done in the new commit.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/20140#discussion_r1689886041