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

Ioi Lam iklam at openjdk.java.net
Tue Dec 8 17:23:17 UTC 2020


On Tue, 8 Dec 2020 16:22:44 GMT, Claes Redestad <redestad at openjdk.org> wrote:

>> 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

Marked as reviewed by iklam (Reviewer).

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

PR: https://git.openjdk.java.net/jdk/pull/1256


More information about the serviceability-dev mailing list