RFR: 8266074: Vtable-based CHA implementation
Vladimir Kozlov
kvn at openjdk.java.net
Fri Apr 30 20:16:11 UTC 2021
On Fri, 30 Apr 2021 19:34:40 GMT, Vladimir Ivanov <vlivanov at openjdk.org> wrote:
>> test/hotspot/jtreg/compiler/cha/StrengthReduceInterfaceCall.java line 35:
>>
>>> 33: *
>>> 34: * @run main/othervm -Xbootclasspath/a:. -XX:+IgnoreUnrecognizedVMOptions -XX:+UnlockDiagnosticVMOptions
>>> 35: * -XX:+UseVtableBasedCHA
>>
>> `UseVtableBasedCHA` is `true` by default. May be duplicate these `runs` for both cases: on and off.
>
> The test assumes `UseVtableBasedCHA` is on and some of the assertions it makes fail with `-XX:-UseVtableBasedCHA`. So, it would require all the changes in the test logic have to be guarded by a check whether UseVtableBasedCHA is on or off.
>
> Considering `UseVtableBasedCHA` is turned on by default in the PR, I don't see much value in complicating the test.
So only new implementation is tested with these changes in test. Got it. May be use:
`@requires vm.opt.final.UseVtableBasedCHA == true`
-------------
PR: https://git.openjdk.java.net/jdk/pull/3727
More information about the hotspot-dev
mailing list