RFR: 8261229: MethodData is not correctly initialized with TieredStopAtLevel=3 [v2]

Mon Feb 8 07:05:42 UTC 2021

On Sat, 6 Feb 2021 20:47:00 GMT, Igor Veresov <iveresov at openjdk.org> wrote:

>> Mostly a typo in compilation mode ergonomics that selected a quick-only mode essentially when the user specified TieredStopAtLevel={1,2,3}. The quick-only mode has an optimization that eliminates parts of the MDO since they are not needed. Meanwhile, the WB API considered it a fair game to request a level 3 compile, that requires a full MDO.
>> 
>> The fix corrects the original issue and also tries to be extra defensive with WB API (since it's semantics is not clearly specified) by always allocating full MDO if WB API is on.
>
> Igor Veresov has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Remove WB defence

Looks good to me.

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

Marked as reviewed by thartmann (Reviewer).

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