RFR: 8373599: Cleanup arguments.hpp includes [v2]
Stefan Karlsson
stefank at openjdk.org
Fri Dec 12 15:06:39 UTC 2025
> In the Valhalla project we are considering using a property in Arguments to figure out if Valhalla is enabled. To minimize transitive-include-proliferation if this property ever needs to be checked in a header I propose that we clean up the header includes in arguments.hpp.
>
> I'm opening the review for this so that I can get opinions on the move of functions. I'm going to compile this on all our platforms and will likely add a few more includes when unrelated files fail to compile because they relied on the includes provided through by java.hpp and arguments.hpp.
Stefan Karlsson has updated the pull request incrementally with one additional commit since the last revision:
Sort order
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/28791/files
- new: https://git.openjdk.org/jdk/pull/28791/files/e1d1bf61..90f3919c
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=28791&range=01
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=28791&range=00-01
Stats: 2 lines in 1 file changed: 1 ins; 1 del; 0 mod
Patch: https://git.openjdk.org/jdk/pull/28791.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/28791/head:pull/28791
PR: https://git.openjdk.org/jdk/pull/28791
More information about the hotspot-dev
mailing list