Integrated: 8367740: assembler_<cpu>.inline.hpp should not include assembler.inline.hpp
Francesco Andreuzzi
fandreuzzi at openjdk.org
Thu Sep 18 09:12:45 UTC 2025
On Tue, 16 Sep 2025 10:15:06 GMT, Francesco Andreuzzi <fandreuzzi at openjdk.org> wrote:
> This is the content of assembler.inline.hpp:
> https://github.com/openjdk/jdk/blob/ca89cd06d39ed3a6bbe16f60fea4d7382849edbd/src/hotspot/share/asm/assembler.inline.hpp#L28-L30
>
> Most of the `assembler_<cpu>.inline.hpp` include it:
> https://github.com/openjdk/jdk/blob/ca89cd06d39ed3a6bbe16f60fea4d7382849edbd/src/hotspot/cpu/zero/assembler_zero.inline.hpp#L29-L32
>
> They should probably include `assembler.hpp` instead.
>
> Testing: tier1 in GHA
This pull request has now been integrated.
Changeset: 4c7c009d
Author: Francesco Andreuzzi <fandreuzzi at openjdk.org>
Committer: Damon Fenacci <dfenacci at openjdk.org>
URL: https://git.openjdk.org/jdk/commit/4c7c009dd6aa2ce1f65f05c05d7376240f3c01cd
Stats: 5 lines in 5 files changed: 0 ins; 0 del; 5 mod
8367740: assembler_<cpu>.inline.hpp should not include assembler.inline.hpp
Reviewed-by: dfenacci, ayang
-------------
PR: https://git.openjdk.org/jdk/pull/27311
More information about the hotspot-compiler-dev
mailing list