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

[Dmitrij Pochepko]

Thank you!

On 21.06.2016 23:17, Vladimir Kozlov wrote:
> 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
>>