The file jvm.h and the various platform-specific jvm_md.h (which are named jvm_<platform>.h in hotspot) have diverted quite a lot from each other between the hotspot version and the java.base version.
There should only be one instance of each of these files.
For what it's worth, it's the java.base version that gets exported with the JRE/JDK image, so a reasonable first assumption is that that version trumps the hotspot version. For jvm.h, it is obvious that there have been a lot of fixes to the java.base version (e.g. corrected typos), but it's not clear that *all* changes in the hotspot jvm.h should go away.
Hotspot has used a special technique for including platform specific files, #include OS_HEADER_H(jvm).
I would consider this bug fixed even if this difference remains -- I'll fix the include/build issues in a separate bug, as long as I know that there are no significant code differences.
There should only be one instance of each of these files.
For what it's worth, it's the java.base version that gets exported with the JRE/JDK image, so a reasonable first assumption is that that version trumps the hotspot version. For jvm.h, it is obvious that there have been a lot of fixes to the java.base version (e.g. corrected typos), but it's not clear that *all* changes in the hotspot jvm.h should go away.
Hotspot has used a special technique for including platform specific files, #include OS_HEADER_H(jvm).
I would consider this bug fixed even if this difference remains -- I'll fix the include/build issues in a separate bug, as long as I know that there are no significant code differences.
- blocks
-
JDK-8167078 Duplicate header files in hotspot and jdk
-
- Resolved
-
- csr for
-
JDK-8190214 Reconcile jvm.h and all jvm_md.h between java.base and hotspot
-
- Closed
-
- is blocked by
-
JDK-8190284 link-time-opt should not be using -fwhole-program
-
- Resolved
-
- relates to
-
JDK-8190617 test/jdk/sun/tools/jhsdb/BasicLauncherTest.java fails
-
- Resolved
-
-
JDK-8190491 SA tests failed after 8189610 changes
-
- Closed
-
-
JDK-8190484 Move jvm.h, jmm.h et al to hotspot/*/include
-
- Resolved
-
-
JDK-8293123 Fix various include file ordering
-
- Resolved
-
(2 relates to)