RFR: 8251462: Remove legacy compilation policy [v3]

Igor Veresov iveresov at openjdk.java.net
Mon Jan 11 17:52:02 UTC 2021


On Mon, 11 Jan 2021 09:06:36 GMT, Boris Ulasevich <bulasevich 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/cpu/arm/templateInterpreterGenerator_arm.cpp line 376:
> 
>> 374:   // Note: In tiered we increment either counters in MethodCounters* or
>> 375:   // in MDO depending if we're profiling or not.
>> 376:   int increment = InvocationCounter::count_increment;
> 
> The small code below seems not to be equivalent replacement for the above code. I would appreciate some explanation on why we change the things. Thanks.

It's a just the code that was guarded before by ```if (TieredCompilation)```. It was just the else part that's gotten removed. Since the new policy is base off of tiered we don't need the old style of callbacks. Is there something else that you've noticed?

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

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


More information about the hotspot-compiler-dev mailing list