RFR: 8256424: Move ciSymbol::symbol_name() to ciSymbols::symbol_name() [v3]

Claes Redestad redestad at openjdk.java.net
Tue Dec 8 16:22:44 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 incrementally with one additional commit since the last revision:

  ciMethod needs classfile/vmIntrinsics.hpp after 8252505

-------------

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/1256/files
  - new: https://git.openjdk.java.net/jdk/pull/1256/files/b8c485b9..2144b16f

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=1256&range=02
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=1256&range=01-02

  Stats: 1 line in 1 file changed: 1 ins; 0 del; 0 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