RFR: 8256424: Move ciSymbol::symbol_name() to ciSymbols::symbol_name()
Ioi Lam
iklam at openjdk.java.net
Tue Dec 8 03:55:13 UTC 2020
On Tue, 17 Nov 2020 12:53:48 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
Changes requested by iklam (Reviewer).
src/hotspot/share/ci/bcEscapeAnalyzer.hpp line 102:
> 100: void compute_escape_info();
> 101: vmIntrinsicID known_intrinsic();
> 102: void compute_escape_for_intrinsic(vmIntrinsicID iid);
I think the use of vmIntrinsics::ID in bcEscapeAnalyzer.cpp should also be changed to vmIntrinsicID for consistency, even though they are the same type. (The same for other hpp files such as ciMethod.hpp)
src/hotspot/share/opto/library_call.cpp line 27:
> 25: #include "precompiled.hpp"
> 26: #include "asm/macroAssembler.hpp"
> 27: #include "ci/ciSymbols.hpp"
This file doesn't seem to use the exports from ciSymbols.hpp.
-------------
PR: https://git.openjdk.java.net/jdk/pull/1256
More information about the hotspot-dev
mailing list