RFR: 8288537: Refactor hotspot/share/memory/iterator.hpp [v2]

Ioi Lam iklam at openjdk.org
Fri Jun 17 15:05:05 UTC 2022


> HotSpot build time has regressed quite a bit since JDK 17 (1m46s vs 2m20s on my machine). Now it's time to do some header file cleanup.
> 
> iterator.hpp contains a hodge-podge of unrelated classes. Some of these classes have dependencies on other headers (e.g., `Devirtualizer` depends on bitMap.hpp) that slow down C++ compilation.
> 
> This patch moves two infrequently used classes, `Devirtualizer` and `SerializeClosure`, to their own header files. This reduces the total number of C++ lines compiled for libjvm.so for about 1%.

Ioi Lam has updated the pull request incrementally with two additional commits since the last revision:

 - moved memory/devirtualizer.hpp -> utilities/devirtualizer.hpp
 - separate out the changes related to SerializeClosure

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/9176/files
  - new: https://git.openjdk.org/jdk/pull/9176/files/ebb83152..159e9530

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=9176&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=9176&range=00-01

  Stats: 208 lines in 26 files changed: 75 ins; 126 del; 7 mod
  Patch: https://git.openjdk.org/jdk/pull/9176.diff
  Fetch: git fetch https://git.openjdk.org/jdk pull/9176/head:pull/9176

PR: https://git.openjdk.org/jdk/pull/9176


More information about the shenandoah-dev mailing list