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