Looks good to me.
--alex
On 07/10/2018 13:37, JC Beyler wrote:
Hi Alex,
Done here:
http://cr.openjdk.java.net/~jcbeyler/8205643/webrev.01/
Any other issues with this fix?
Thanks!
Jc
On Tue, Jul 10, 2018 at 12:26 PM Alex Menkov <alexey.men...@oracle.com
<mailto:alexey.men...@oracle.com>> 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
