[9] Review request : JDK-8067005 : Several java/lang/invoke tests fail due to exhausted code cache

Wed Jun 17 16:50:40 UTC 2015

Looks good.

Best regards,
Vladimir Ivanov

On 6/17/15 7:23 PM, Konstantin Shefov wrote:
> Vladimir,
>
> I have updated the webrev:
> http://cr.openjdk.java.net/~kshefov/8067005/webrev.01/
>
> -Konstantin
>
> On 06/17/2015 04:01 PM, Vladimir Ivanov wrote:
>> Konstantin,
>>
>> I'd move isThrowableCausedByVME & processVME from Utils to a more
>> specific place (new shared class for JSR292 tests?). They aren't
>> general enough to be useful for other tests.
>>
>> Also, processVME name is confusing. I'd introduce something like a
>> runMHTest wrapper and process exceptions there.
>>
>> Best regards,
>> Vladimir Ivanov
>>
>> On 6/17/15 1:19 PM, Konstantin Shefov wrote:
>>> Hello,
>>>
>>> Please review the test bug fix
>>> https://bugs.openjdk.java.net/browse/JDK-8067005
>>> Webrev is http://cr.openjdk.java.net/~kshefov/8067005/webrev.00/
>>>
>>> Thanks
>>> -Konstantin
>