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