RFR: 8256424: Move ciSymbol::symbol_name() to ciSymbols::symbol_name()
Claes Redestad
redestad at openjdk.java.net
Sat Dec 5 13:18:20 UTC 2020
This moves the mirroring of vmSymbols in ciSymbols to a separate file, ciSymbols.hpp, to reduce includes throughout hotspot (and clean up the ciSymbol namespace). In a few places code is moved from .hpp to .cpp to facilitate this.
With PCH disabled, this reduces total includes from 276949 to 276332
-------------
Commit messages:
- remote merge
- Prepare for 1237 changes
- Adjust includes after vmIntrinsic changes
- Merge master
- Outline is_call_site_target to remove include from ciField.hpp
- Outline is_autobox_cache to remove include from ciField.hpp
- Merge branch 'master' into ciSymbols
- Fix bad definition
- Define debug-only sid_ok in .cpp to avoid need to include vmSymbols.hpp in ciSymbol.hpp
- Revert accidental include order swap
- ... and 1 more: https://git.openjdk.java.net/jdk/compare/86b65756...b05957b8
Changes: https://git.openjdk.java.net/jdk/pull/1256/files
Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=1256&range=00
Issue: https://bugs.openjdk.java.net/browse/JDK-8256424
Stats: 174 lines in 32 files changed: 102 ins; 30 del; 42 mod
Patch: https://git.openjdk.java.net/jdk/pull/1256.diff
Fetch: git fetch https://git.openjdk.java.net/jdk pull/1256/head:pull/1256
PR: https://git.openjdk.java.net/jdk/pull/1256
More information about the serviceability-dev
mailing list