RFR: 8310076: Reduce inclusion of bytecodeStream.hpp

Ioi Lam iklam at openjdk.org
Thu Jun 15 01:23:54 UTC 2023


The popular header file generateOopMap.hpp includes bytecodeStream.hpp, but it doesn't actually needs the full declaration of the BytecodeStream class.

This inclusion can be replaced with a simple forward declaration of the BytecodeStream class.

This reduces the number of HotSpot .o files that include bytecodeStream.hpp from 571 to 21.

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

Commit messages:
 - 8310076: Reduce inclusion of bytecodeStream.hpp

Changes: https://git.openjdk.org/jdk/pull/14481/files
 Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=14481&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8310076
  Stats: 13 lines in 7 files changed: 9 ins; 1 del; 3 mod
  Patch: https://git.openjdk.org/jdk/pull/14481.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/14481/head:pull/14481

PR: https://git.openjdk.org/jdk/pull/14481


More information about the hotspot-dev mailing list