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