RFR: 8280469: C2: CHA support for interface calls when inlining through method handle linker [v2]

Vladimir Kozlov kvn at openjdk.org
Fri Nov 7 22:54:01 UTC 2025


On Fri, 7 Nov 2025 22:45:53 GMT, Vladimir Kozlov <kvn at openjdk.org> wrote:

>>> Should you check that C2 is enabled?
>> 
>> The test has `@requires !vm.graal.enabled`. Do you prefer to have it spelled as `@requires vm.compiler2.enabled` instead?
>> 
>>> May it should be in @requires?
>> 
>> Original test cases apply to both C1 and C2. I could introduce a separate test for MH invoker cases, but IMO keeping relevant test logic co-located is preferred compared to avoiding a configuration check at runtime.
>
> ok

We can have VM build without C2 or only C1 is enabled (TieredStopAtLevel flag).
But based on your response the test should work even when only C1 is used. So it is fine.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/28094#discussion_r2505779693


More information about the hotspot-compiler-dev mailing list