> On Dec 27, 2016, at 2:40 AM, Erik Joelsson <erik.joels...@oracle.com> wrote: > > The new module, jdk.vm.compiler (as well as jdk.vm.ci), requires access to > some of the newly built class files in order to run an annotation processor > in the gensrc step. The extra dependency declaration for this is currently > missing one module (java.instrument) in make/Main.gmk. To avoid having to > maintain the list of modules they depend on in two places (module-info.java > and the makefile), we can generate the dependency targets in the makefile, > since we already have the module dependency information in make. > > Bug: https://bugs.openjdk.java.net/browse/JDK-8171929 > > Webrev: http://cr.openjdk.java.net/~erikj/8171929/webrev.01/
This looks okay to me. Mandy
