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


Reply via email to