Pre-RFR (L) 8199263: Split interfaceSupport.hpp to not require including .inline.hpp files

coleen.phillimore at oracle.com coleen.phillimore at oracle.com
Tue Mar 13 11:50:12 UTC 2018


Summary: interfaceSupport.hpp is an inline file so moved to 
interfaceSupport.inline.hpp and stopped including it in .hpp files

90% of this change is renaming interfaceSupport.hpp to 
interfaceSupport.inline.hpp.   I tried to see if all of these files 
needed this header and the answer was yes.   A surprising (to me!) 
number of files have thread state transitions.
Some of interesting part of this change is adding ciUtilities.inline.hpp 
to include interfaceSupport.inline.hpp for VM_ENTRY.  
whitebox.inline.hpp was added for the same reason.
jvmtiEnter.hpp was renamed jvmtiEnter.inline.hpp because it includes 
interfaceSupport.inline.hpp, and is only included in cpp files.
The rest of the changes were to add back includes that are not pulled in 
by header files including interfaceSupport.hpp, like gcLocker.hpp and of 
course handles.inline.hpp.

This probably overlaps some of Volker's patch.  Can this be tested on 
other platforms that we don't have?

Hopefully, at the end of all this we have more clean header files so 
that transitive includes don't make the jvm build on one platform but 
not the next.  I think that's the goal of all of this work.

This was tested with Oracle platforms (linux-x64, solaris-sparcv9, 
macosx-x64, windows-x64) in the mach5 tier1 and 2.   I built this 
locally without precompiled headers (my default setting of course) on 
linux-x64.

bug link https://bugs.openjdk.java.net/browse/JDK-8199263
local webrev at 
http://oklahoma.us.oracle.com/~cphillim/webrev/8199263.02/webrev

Thanks to Stefan for his help with this.

Thanks,
Coleen




More information about the hotspot-dev mailing list