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:49:07 UTC 2025
On Tue, 4 Nov 2025 19:34:54 GMT, Vladimir Ivanov <vlivanov at openjdk.org> wrote:
>> May it should be in @requires ?
>
>> 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
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/28094#discussion_r2505773293
More information about the hotspot-compiler-dev
mailing list