RFR: 8059551: JEP-JDK-8043304: Test task: stress tests

[Pavel Chistyakov]

Hi Christian,

Yes, here is updated version: http://cr.openjdk.java.net/~iignatyev/pchistyakov/8059551/webrev.03
Could you please take a look into it?

--------
Thanks,
Pavel

----- Original Message -----
From: christian.thalinger at oracle.com
To: igor.ignatyev at oracle.com
Cc: pavel.chistyakov at oracle.com, hotspot-compiler-dev at openjdk.java.net
Sent: Friday, December 19, 2014 1:17:48 AM GMT +04:00 Abu Dhabi / Muscat
Subject: Re: RFR: 8059551: JEP-JDK-8043304: Test task: stress tests


> On Dec 15, 2014, at 5:34 AM, Igor Ignatyev <igor.ignatyev at oracle.com> wrote:
> 
> Pavel,
> 
> http://cr.openjdk.java.net/~iignatyev/pchistyakov/8059551/webrev.02/test/compiler/codecache/stress/Helper.java.html 
> 
>> 45             CLASS_DATA = loadClassData(TestCaseImpl.class.getName());
>> 95             String className = TestCaseImpl.class.getName();
> 
> these lines will load 'TestCaseImpl' class via default class loader, could you try to avoid it?

Any update on that?

> 
> Igor
> 
> On 12/15/2014 04:15 PM, Pavel Chistyakov wrote:
>> Hi,
>> 
>> please take a look into slightly updated webrev:
>> http://cr.openjdk.java.net/~iignatyev/pchistyakov/8059551/webrev.02/
>> <http://cr.openjdk.java.net/%7Eiignatyev/pchistyakov/8059551/webrev.02/>
>> 
>> --------------
>> Thanks,
>> Pavel
>> 
>> On 12.12.2014 23:05, Vladimir Kozlov wrote:
>>> Looks fine.
>>> 
>>> Thanks,
>>> Vladimir
>>> 
>>> On 12/12/14 6:07 AM, David Chase wrote:
>>>> 
>>>> On 2014-12-12, at 5:14 AM, Pavel Chistyakov
>>>> <pavel.chistyakov at oracle.com> wrote:
>>>> 
>>>>> David, thank you for review.
>>>>> Here is updated webrev:
>>>>> http://cr.openjdk.java.net/~iignatyev/pchistyakov/8059551/webrev.01
>>>>> <http://cr.openjdk.java.net/%7Eiignatyev/pchistyakov/8059551/webrev.01>
>>>> 
>>>> That is excellent -- now we just need a real reviewer.
>>>> 
>>>> David
>>>> 
>>