RFR: 8251462: Remove legacy compilation policy [v3]

Igor Veresov iveresov at openjdk.java.net
Fri Jan 22 18:22:09 UTC 2021


On Fri, 22 Jan 2021 01:11:33 GMT, Vladimir Kozlov <kvn at openjdk.org> wrote:

>> Igor Veresov has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Fix another s390 compilation failure
>
> src/hotspot/share/jvmci/vmStructs_jvmci.cpp line 229:
> 
>> 227:   JVMTI_ONLY(nonstatic_field(MethodCounters,   _number_of_breakpoints,                        u2))                                   \
>> 228:   nonstatic_field(MethodCounters,              _invocation_counter,                           InvocationCounter)                     \
>> 229:   nonstatic_field(MethodCounters,              _backedge_counter,                             InvocationCounter)                     \
> 
> I don't see Java JVMCI changes. Do you need them?

I haven't found any uses of it.

> test/hotspot/jtreg/TEST.quick-groups line 1787:
> 
>> 1785:   vmTestbase/jit/t/t112/t112.java \
>> 1786:   vmTestbase/jit/t/t113/t113.java \
>> 1787:   vmTestbase/jit/verifier/VerifyInitLocal/VerifyInitLocal.java \
> 
> Why this test removed?

It was an odd test that check if the -XX:+PrintTieredEvents flag didn't work with -XX:-TieredCompilation. Since now it always works that test doesn't make much sense anymore.

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

PR: https://git.openjdk.java.net/jdk/pull/1985


More information about the hotspot-compiler-dev mailing list