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
