|
Hi Jc,
I'll test and push it.
Thanks,
Serguei
On 7/10/18 15:31, JC Beyler wrote:
Hi Serguei,
Here it is:
Could someone test/push it please?
Thanks!
Jc
Sorry, did not see your reply to
Alex.
Looks good - ship it!
Thanks,
Serguei
On 7/10/18 14:54, serguei.spit...@oracle.com
wrote:
> Hi Jc,
>
> The fix looks good.
> Alex is right.
> I forgot to tell you that the test has be excluded from
the file:
> open/test/hotspot/jtreg/ProblemList.txt
>
> Thanks,
> Serguei
>
>
> On 7/10/18 12:26, Alex Menkov wrote:
>> Hi JC,
>>
>> you need also to remove the test from ProblemList
>>
>> --alex
>>
>> On 07/10/2018 11:41, JC Beyler wrote:
>>> Hi All,
>>>
>>> Could someone review the one liner for the bug:
>>> https://bugs.openjdk.java.net/browse/JDK-8205643
>>>
>>> The webrev is here:
>>> http://cr.openjdk.java.net/~jcbeyler/8205643/webrev.00/
>>>
>>> Basically, the test is testing CMS and Graal does
not play well with
>>> CMS it seems so this removes Graal being tested
with it.
>>>
>>> Thanks,
>>> Jc
>
--
|
