Integrated: JDK-8280503: Use allStatic.hpp instead of allocation.hpp where possible
Thomas Stuefe
stuefe at openjdk.java.net
Tue Jan 25 09:18:44 UTC 2022
On Sat, 22 Jan 2022 13:33:24 GMT, Thomas Stuefe <stuefe at openjdk.org> wrote:
> JDK-8249944 moved AllStatic to its own header. We should use that one instead of allocation.hpp where possible to reduce header dependencies.
>
> This patch:
> - replaces includes of allocation.hpp with allstatic.hpp where appropiate
> - fixes up resulting errors since this changes uncovers missing dependencies. Mainly, missing includes of debug.hpp, of globalDefinitions.hpp, and missing outputStream definitions.
>
> Changes are trivial but onerous. Done partly with a script, partly manually.
>
> Test:
> - Checked the build with gtests on Linux x86, x64, minimal, zero, aarch64, for both fastdebug and release. All builds of course without PCH.
> - GHAs
This pull request has now been integrated.
Changeset: 2155afe2
Author: Thomas Stuefe <stuefe at openjdk.org>
URL: https://git.openjdk.java.net/jdk/commit/2155afe2a87d718757b009d712361d7a63946a7f
Stats: 368 lines in 172 files changed: 35 ins; 0 del; 333 mod
8280503: Use allStatic.hpp instead of allocation.hpp where possible
Reviewed-by: dholmes, iklam
-------------
PR: https://git.openjdk.java.net/jdk/pull/7188
More information about the serviceability-dev
mailing list