RFR: 8260306: Do not include osThread.hpp in thread.hpp

Ioi Lam iklam at openjdk.java.net
Fri Jan 22 18:14:45 UTC 2021


thread.hpp is a popular header file (included by 856 out of 1000 HotSpot.o file), so we should try to minimize what files are included by thread.hpp.

We can replace the inclusion of osThread.hpp with a forward declaration of the OSThread class.

This reduces the number of .o files that include osThread.hpp to 42.

Tested with mach5: tier1, builds-tier2, builds-tier3, builds-tier4 and builds-tier5. Also locally: aarch64, arm, ppc64, s390, x86, and zero.

-------------

Commit messages:
 - 8260306: Do not include osThread.hpp in thread.hpp

Changes: https://git.openjdk.java.net/jdk/pull/2198/files
 Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=2198&range=00
  Issue: https://bugs.openjdk.java.net/browse/JDK-8260306
  Stats: 31 lines in 19 files changed: 19 ins; 1 del; 11 mod
  Patch: https://git.openjdk.java.net/jdk/pull/2198.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/2198/head:pull/2198

PR: https://git.openjdk.java.net/jdk/pull/2198


More information about the hotspot-dev mailing list