RFR: 8354309: Sort GC includes [v2]

Stefan Karlsson stefank at openjdk.org
Wed Apr 23 10:10:20 UTC 2025


> We now have a tool to fix our include lines to better adhere to the HotSpot Style Guide. See test/hotspot/jtreg/sources/SortIncludes.java. I've run the tool against the GC code and propose the resulting changes. I did one manual change to move a section guarded with INCLUDE_JFR.
> 
> Builds successfully with GHA. I will run this through tier1-2.

Stefan Karlsson 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:

 - Fix windows compliation error
 - Merge remote-tracking branch 'upstream/master' into 8354309_sort_gc_includes
 - Merge remote-tracking branch 'upstream/master' into 8354309_sort_gc_includes
 - 8354309: Sort GC includes

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/24581/files
  - new: https://git.openjdk.org/jdk/pull/24581/files/cf81077d..54a5d61b

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

  Stats: 214494 lines in 1181 files changed: 38563 ins; 172575 del; 3356 mod
  Patch: https://git.openjdk.org/jdk/pull/24581.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/24581/head:pull/24581

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


More information about the hotspot-dev mailing list