RFR: 8310494: Do not include constantPool.hpp from instanceKlass.hpp [v3]
Ioi Lam
iklam at openjdk.org
Thu Jun 22 22:38:12 UTC 2023
> instanceKlass.hpp is needed quite frequently for HotSpot C++ sources, but constantPool.hpp is much less frequently.
>
> This PR is the first step of reducing the inclusion of constantPool.hpp, by removing it from instanceKlass.hpp
>
> I moved a bunch of inline functions into instanceKlass.cpp. These don't seem to be performance related.
Ioi Lam has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains four additional commits since the last revision:
- Merge branch 'master' into 8310494-remove-constantPool-from-instanceKlass-hpp
- moved comment about on_stack() to the cpp file (which already has the comment)
- @stefank review comments
- 8310494: Do not include constantPool.hpp from instanceKlass.hpp
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/14597/files
- new: https://git.openjdk.org/jdk/pull/14597/files/68697195..f8bd76ac
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=14597&range=02
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=14597&range=01-02
Stats: 2078 lines in 190 files changed: 1087 ins; 325 del; 666 mod
Patch: https://git.openjdk.org/jdk/pull/14597.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/14597/head:pull/14597
PR: https://git.openjdk.org/jdk/pull/14597
More information about the hotspot-dev
mailing list