RFR: 8256424: Move ciSymbol::symbol_name() to ciSymbols::symbol_name() [v2]
Claes Redestad
redestad at openjdk.java.net
Tue Dec 8 13:22:22 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
Claes Redestad has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 13 commits:
- Merge branch 'master' into ciSymbols
- @iklam reviews, additional cleanup
- 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
- ... and 3 more: https://git.openjdk.java.net/jdk/compare/d0c52651...b8c485b9
-------------
Changes: https://git.openjdk.java.net/jdk/pull/1256/files
Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=1256&range=01
Stats: 182 lines in 32 files changed: 101 ins; 30 del; 51 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