On Fri, 22 Jan 2021 01:11:33 GMT, Vladimir Kozlov <k...@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

Reply via email to