Integrated: 8373599: Cleanup arguments.hpp includes
Stefan Karlsson
stefank at openjdk.org
Mon Dec 15 12:59:50 UTC 2025
On Fri, 12 Dec 2025 14:11:55 GMT, Stefan Karlsson <stefank at openjdk.org> wrote:
> 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.
This pull request has now been integrated.
Changeset: f5187ebf
Author: Stefan Karlsson <stefank at openjdk.org>
URL: https://git.openjdk.org/jdk/commit/f5187ebf7a4d4241f01612b62c514a1e4e272658
Stats: 67 lines in 7 files changed: 37 ins; 19 del; 11 mod
8373599: Cleanup arguments.hpp includes
Reviewed-by: coleenp, kbarrett
-------------
PR: https://git.openjdk.org/jdk/pull/28791
More information about the hotspot-dev
mailing list