I've updated the webrev with a fix for another regression caused by JDK-8187490:
--- old/test/jdk/jdk/modules/etc/UpgradeableModules.java 2018-04-18
12:57:32.000000000 +0200
+++ new/test/jdk/jdk/modules/etc/UpgradeableModules.java 2018-04-18
12:57:32.000000000 +0200
@@ -46,6 +46,7 @@
List.of("java.compiler",
"java.jnlp",
"jdk.internal.vm.compiler",
+ "jdk.internal.vm.compiler.management",
"jdk.deploy",
"jdk.javaws",
"jdk.plugin",
-Doug
> On 18 Apr 2018, at 12:35, Doug Simon <doug.si...@oracle.com> wrote:
>
> Right after pushing the change for JDK-8187490, I noticed that the mach5
> build had 2 CTW test failures due to jdk.internal.vm.compiler.management now
> being an empty module. This will be fixed in the next Graal update but the
> failing tests should be fixed in the meantime.
>
> http://cr.openjdk.java.net/~dnsimon/8201794/
> https://bugs.openjdk.java.net/browse/JDK-8201794
>
> I'm running mach5 now to ensure the regressions are fixed and no other
> regressions occur.
>
> -Doug
