RFR(S) 8251560 Remove excessive header file inclusion from systemDictionary.hpp and others
Coleen Phillimore
coleen.phillimore at oracle.com
Fri Aug 14 18:35:12 UTC 2020
I really like this change. It looks like you need a classLoadInfo.hpp
file with the constructors.
Thanks,
Coleen
On 8/14/20 12:54 PM, Ioi Lam wrote:
> https://bugs.openjdk.java.net/browse/JDK-8251560
> http://cr.openjdk.java.net/~iklam/jdk16/8251560-remove-includes-in-system-dict-etc.v01/
>
>
> Summary:
>
> I removed a few unnecessary #includes from systemDictionary.hpp,
> signature.hpp, altHashing.hpp and klass.inline.hpp
>
> Two rarely used classes are moved to a new header file classLoadInfo.hpp
>
> Some non-critical inline functions were moved to cpp files.
>
> Some cpp files needed to be updated (because they didn't explicitly
> declare dependencies on header files that were transitively included
> by other header files).
>
> Tested with mach5 tier1 and also local Linux builds (no-pch, minimal
> VM, etc).
> Thanks
> - Ioi
More information about the hotspot-runtime-dev
mailing list