RFR: 8264868: Reduce inclusion of registerMap.hpp and register.hpp [v2]
Ioi Lam
iklam at openjdk.java.net
Fri Apr 9 23:28:04 UTC 2021
> Register.hpp is included 815 times, and registerMap.hpp is include 862 times (out of about 1000 HotSpot .o files).
>
> This can be reduced by refactoring the popular header file **frame.hpp**, so that it doesn't include registerMap.hpp anymore. This reduces the number of .o files that include register.hpp to 612, and that of registerMap.hpp to 109.
>
> The total number of lines of C++ code compiled for HotSpot is reduced by about 0.5%.
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 two additional commits since the last revision:
- Merge branch 'master' into 8264868-reduce-registerMap-hpp
- 8264868: Reduce inclusion of registerMap.hpp and register.hpp
-------------
Changes:
- all: https://git.openjdk.java.net/jdk/pull/3384/files
- new: https://git.openjdk.java.net/jdk/pull/3384/files/507136ef..288658cf
Webrevs:
- full: https://webrevs.openjdk.java.net/?repo=jdk&pr=3384&range=01
- incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=3384&range=00-01
Stats: 6195 lines in 169 files changed: 4507 ins; 1090 del; 598 mod
Patch: https://git.openjdk.java.net/jdk/pull/3384.diff
Fetch: git fetch https://git.openjdk.java.net/jdk pull/3384/head:pull/3384
PR: https://git.openjdk.java.net/jdk/pull/3384
More information about the hotspot-dev
mailing list