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