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