Hi Magnus,

On 4/12/2017 9:06 PM, Magnus Ihse Bursie wrote:
JDK-8190484 was created as a follow-up bug to the unification of the duplicated jvm.h, jvm_md.h and jmm.h, to determine the proper location of these files. This has now been decided to be hotspot/share/include and hotspot/os/$OS/include, respectively.

This patch moves the relevant files there, but since this also frees up the src/$MODULE/native/include directories for the original purpose, it also unifies and simplifies the build logic for these directories, so that common code is executed for all modules to just copy any exported header files from these directories, should they exist.

I'm intending to push this to jdk-hs.

Do you want this in 10 or 11? If you want 10 then you need to push to jdk/jdk as there may not be another snapshot from jdk/hs.

Bug: https://bugs.openjdk.java.net/browse/JDK-8190484
WebRev: http://cr.openjdk.java.net/~ihse/JDK-8190484-move-hotspot-exported-includes/webrev.01

Seems okay for the jvm.h/jmm.h changes. I couldn't follow all the flow on cleanups.

Thanks,
David


/Magnus

Reply via email to