RFR: 8292007: Do not include vmSymbol.hpp in method.hpp

Ioi Lam iklam at openjdk.org
Mon Aug 8 22:14:25 UTC 2022


A small build speed improvement:

The popular header method.hpp has this inline function


bool is_continuation_enter_intrinsic() const {
  return intrinsic_id() == vmIntrinsics::_Continuation_enterSpecial;
}


which causes the large header files vmSymbols.hpp and vmIntrinsics.hpp to be unnecessarily included by many HotSpot object files.

The fix is to move this function to method.inline.hpp.

(It might be performance critical since it's used by some loops that walk the call frames).

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

Commit messages:
 - 8292007: Do not include vmSymbol.hpp in method.hpp

Changes: https://git.openjdk.org/jdk/pull/9789/files
 Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=9789&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8292007
  Stats: 25 lines in 13 files changed: 18 ins; 2 del; 5 mod
  Patch: https://git.openjdk.org/jdk/pull/9789.diff
  Fetch: git fetch https://git.openjdk.org/jdk pull/9789/head:pull/9789

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


More information about the hotspot-dev mailing list