Hi Serguei, Here it is: http://cr.openjdk.java.net/~jcbeyler/8205643/webrev.02/
Could someone test/push it please? Thanks! Jc On Tue, Jul 10, 2018 at 2:56 PM serguei.spit...@oracle.com < serguei.spit...@oracle.com> wrote: > 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 > > > > -- Thanks, Jc
