RFR(S) Remove excessive inclusion of arguments.hpp
Thomas Stüfe
thomas.stuefe at gmail.com
Sun Aug 23 08:31:18 UTC 2020
Hi Ioi,
looks okay, but why the added java.hpp includes? At first glance does not
seem related to your bug description.
Thanks Thomas
On Fri, Aug 21, 2020 at 6:47 PM Ioi Lam <ioi.lam at oracle.com> wrote:
> https://bugs.openjdk.java.net/browse/JDK-8252151
>
> http://cr.openjdk.java.net/~iklam/jdk16/8252151-dont-include-arguments-hpp.v01/
>
> The large header file arguments.hpp is included by popular header files
> such as constMethod.hpp and methodCounters.hpp (which are included by
> method.hpp, which is included by almost every cpp file).
>
> The dependencies on arguments.hpp by constMethod.hpp are very small, and
> can be moved to constMethod.cpp.
>
> methodCounters.hpp does not depend on arguments.hpp
>
> ===
> by removing these dependencies, we can reduce the total number of
> include files in the hotspot build from 530,921 to 523,010, or a
> reduction of about 1.5%.
>
>
>
More information about the hotspot-runtime-dev
mailing list