RFR 8221698 Remove redundant includes from popular header files

Thomas Stüfe thomas.stuefe at gmail.com
Fri Mar 29 20:12:41 UTC 2019

Hi Ioi,

thanks, good cleanup! I think we could do a lot more like these.

Looks good to me, if you remove the commented-out lines in
instanceKlass.hpp as Coleen mentioned.

I do not need another webrev.

Cheers, Thomas

On Fri, Mar 29, 2019 at 5:36 PM Ioi Lam <ioi.lam at oracle.com> wrote:

> https://bugs.openjdk.java.net/browse/JDK-8221698
> http://cr.openjdk.java.net/~iklam/jdk13/8221698-remove-redundant-includes.v01/
> The following files are included by some popular header files. This
> causes them to be included by almost all HotSpot C files. However, these
> header are for features that are not commonly used. Excluding them can
> improve HotSpot build speed by 1~2%.
>    histogram.hpp
>    classLoader.hpp
>    utf8.hpp
>    moduleEntry.hpp
>    packageEntry.hpp
> Thanks
> - Ioi

More information about the hotspot-runtime-dev mailing list