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