RFR: 8364359: Sort share/cds includes [v2]
Aleksey Shipilev
shade at openjdk.org
Thu Jul 31 12:33:56 UTC 2025
On Wed, 30 Jul 2025 17:43:09 GMT, Francesco Andreuzzi <duke at openjdk.org> wrote:
>> This PR sorts the includes in `hotspot/share/cds` using `SortIncludes.java`. I'm also adding the directory to `TestIncludesAreSorted`.
>>
>> Passes tier1.
>
> Francesco Andreuzzi has updated the pull request incrementally with one additional commit since the last revision:
>
> system includes below
Looks good to me with a minor nit. But I have missed things before, so someone else needs to take a look as well. For example @iklam or @calvinccheung, who are often in this area.
test/hotspot/jtreg/sources/TestIncludesAreSorted.java line 49:
> 47: "share/c1",
> 48: "share/ci",
> 49: "share/cds",
Unordered: `cds` < `ci`?
-------------
Marked as reviewed by shade (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/26561#pullrequestreview-3075203723
PR Review Comment: https://git.openjdk.org/jdk/pull/26561#discussion_r2245250011
More information about the hotspot-runtime-dev
mailing list