RFR: 8264748: Do not include arguments.hpp from compilerDefinitions.hpp

Ioi Lam iklam at openjdk.java.net
Tue Apr 6 15:54:44 UTC 2021


compilerDefinitions.hpp is a popular header, included by 588 of about 1000 HotSpot .o files. It includes arguments.hpp only for a single function that's not used in critical paths.

After refactoring, the number of .o files that includes arguments.hpp reduces from 623 to 68.

(I also removed arguments.hpp from other files that don't actually need it.)

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

Commit messages:
 - 8264748: Do not include arguments.hpp from compilerDefinitions.hpp

Changes: https://git.openjdk.java.net/jdk/pull/3351/files
 Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=3351&range=00
  Issue: https://bugs.openjdk.java.net/browse/JDK-8264748
  Stats: 59 lines in 41 files changed: 16 ins; 33 del; 10 mod
  Patch: https://git.openjdk.java.net/jdk/pull/3351.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/3351/head:pull/3351

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


More information about the hotspot-dev mailing list