RFR(S): 8159730 - compiler/jvmci/compilerToVM/IsMatureTest failed with "Multiple times invoked method should have method data (assert failed: 0 != 0)"

[Vladimir Kozlov]

Looks good.

Thanks,
Vladimir

On 6/21/16 6:17 AM, Dmitrij Pochepko wrote:
> Hi,
>
> I've prepared v02 with slightly another wording in comment to improve readability:
> http://cr.openjdk.java.net/~dpochepk/8159730/webrev.02/
>
> Thanks,
> Dmitrij
>> Hi,
>>
>> please review fix for 8159730 - compiler/jvmci/compilerToVM/IsMatureTest failed with "Multiple times invoked method
>> should have method data (assert failed: 0 != 0)"
>>
>> This test sometimes fails because it expects test method to have method data object even in case it's compiled at
>> level 1(or even running in interpreter mode).
>>
>> I've added guard to disable last checks for compilation level 0 and 1.
>>
>> webrev: http://cr.openjdk.java.net/~dpochepk/8159730/webrev.01/
>>
>> (somehow webrev haven't saved indentation, so, it's probably easier to look at diff directly:
>> http://cr.openjdk.java.net/~dpochepk/8159730/webrev.01/hotspot.patch)
>>
>> CR: https://bugs.openjdk.java.net/browse/JDK-8159730
>>
>> I've tested these changes locally on linux-x64 (including -XX:TieredStopAtLevel=1 mode).
>>
>> Thanks,
>> Dmitrij
>