RFR: 8366556: Sort share/runtime includes

Albert Mingkun Yang ayang at openjdk.org
Mon Sep 1 13:16:57 UTC 2025


On Mon, 1 Sep 2025 12:49:20 GMT, Francesco Andreuzzi <duke at openjdk.org> wrote:

> This PR sorts the includes in `hotspot/share/runtime` using `SortIncludes.java`, and removes some unnecessary ones. I'm also adding the directory to `TestIncludesAreSorted`.
> 
> Passes tier1.

A question regarding preexisting include; otherwise good.

src/hotspot/share/runtime/flags/jvmFlagLimit.cpp line 26:

> 24: 
> 25: #include "gc/shared/jvmFlagConstraintsGC.hpp"
> 26: #include "gc/shared/referenceProcessor.hpp"

Why is `referenceProcessor` needed in this file?

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

Marked as reviewed by ayang (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/27034#pullrequestreview-3173626893
PR Review Comment: https://git.openjdk.org/jdk/pull/27034#discussion_r2313931861


More information about the hotspot-dev mailing list