[lworld] RFR: 8367245: [lworld] C2 compilation fails with "Missed optimization opportunity in PhaseIterGVN" [v2]

[Marc Chevalier]

On Thu, 13 Nov 2025 08:04:12 GMT, Christian Hagedorn <chagedorn at openjdk.org> wrote:

>> Marc Chevalier has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   addFlags
>
> test/hotspot/jtreg/compiler/valhalla/inlinetypes/TestArrays.java line 141:
> 
>> 139:  *          java.base/jdk.internal.vm.annotation
>> 140:  * @requires (os.simpleArch == "x64" | os.simpleArch == "aarch64")
>> 141:  * @run main/othervm -XX:+IgnoreUnrecognizedVMOptions -XX:VerifyIterativeGVN=000 compiler.valhalla.inlinetypes.TestArrays 6
> 
> That looks reasonable to do.
> 
> If we are only seeing the failures in the test VMs, then an easier solution would be to directly add the flag to the framework:
> 
>  InlineTypes.getFramework()
>                    .addScenarios(scenarios[Integer.parseInt(args[0])])
>                    .addFlags("-XX:+IgnoreUnrecognizedVMOptions -XX:VerifyIterativeGVN=000")
>                    .addHelperClasses(MyValue1.class, MyValue2.class, MyValue2Inline.class)
>                    .start();

Like so?

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

PR Review Comment: https://git.openjdk.org/valhalla/pull/1732#discussion_r2523515079