RFR: 8342588: Shenandoah: Pull shared closures together in one header [v3]

Aleksey Shipilev shade at openjdk.org
Tue Oct 29 18:18:01 UTC 2024


> Currently we have shenandoahClosures.hpp and shenandoahOopClosures.hpp, each of which have some closure inside. There is no reason to have this separation, and we should really just merge everything in one header. I also did a bit of mechanical renames. There should be no semantic change with this patch.
> 
> Additional testing:
>  - [x] Linux x86_64 server fastdebug, `hotspot_gc_shenandoah`

Aleksey Shipilev 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 three additional commits since the last revision:

 - Merge branch 'master' into JDK-8342588-shenandoah-shared-closures
 - Fix Zero builds
 - Fix

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/21579/files
  - new: https://git.openjdk.org/jdk/pull/21579/files/c9af382c..831372cf

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

  Stats: 44445 lines in 716 files changed: 21494 ins; 20581 del; 2370 mod
  Patch: https://git.openjdk.org/jdk/pull/21579.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/21579/head:pull/21579

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


More information about the hotspot-gc-dev mailing list