RFR: 8261672: Reduce inclusion of classLoaderData.hpp
Ioi Lam
iklam at openjdk.java.net
Fri Feb 12 19:53:52 UTC 2021
classLoaderData.hpp is included by about 700 out of 1000 .o files in HotSpot. Most of these are transitively included through klass.hpp, typeArrayKlass.hpp and instanceKlass.hpp.
These headers can be refactored by moving inline functions that depend on ClassLoaderData to xxx.inline.hpp. This reduces the .o files that include classLoaderData.hpp to about 260.
(I also removed a bunch of unnecessary inclusion of classLoader.hpp from a few C files).
Tested with mach5: tier1, builds-tier2, builds-tier3, builds-tier4 and builds-tier5. Also locally: aarch64, arm, ppc64, s390, x86, and zero.
-------------
Commit messages:
- 8261672: Reduce inclusion of classLoaderData.hpp
- reduce classLoader.hpp
Changes: https://git.openjdk.java.net/jdk/pull/2555/files
Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=2555&range=00
Issue: https://bugs.openjdk.java.net/browse/JDK-8261672
Stats: 160 lines in 59 files changed: 108 ins; 47 del; 5 mod
Patch: https://git.openjdk.java.net/jdk/pull/2555.diff
Fetch: git fetch https://git.openjdk.java.net/jdk pull/2555/head:pull/2555
PR: https://git.openjdk.java.net/jdk/pull/2555
More information about the hotspot-dev
mailing list