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