RFR: 8265103: Remove unnecessary inclusion of oopMap.hpp

David Holmes dholmes at openjdk.java.net
Tue Apr 13 04:08:58 UTC 2021


On Tue, 13 Apr 2021 01:55:13 GMT, Ioi Lam <iklam at openjdk.org> wrote:

> oopMap.hpp is included by about 585 out of about 1000 .o files in HotSpot. In turn, it pulls in other complex headers such as vmreg.hpp, register.hpp and their cpu-dependent headers.
> 
> There are only 4 header files that include oopMap.hpp. All of these can be replaced for forward declarations.
> 
> According to https://github.com/iklam/tools/tree/main/headers
> 
> 
> [before]

Seems okay.

Thanks,
David

-------------

Marked as reviewed by dholmes (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/3446


More information about the shenandoah-dev mailing list