Integrated: 8256955: Move includes of events.hpp out of header files

Stefan Karlsson stefank at openjdk.java.net
Thu Jan 14 11:38:01 UTC 2021


On Wed, 13 Jan 2021 09:51:12 GMT, Stefan Karlsson <stefank at openjdk.org> wrote:

> I was making changes to events.hpp and saw that the majority of cpp files were recompiled. There are two header files that include events.hpp. I propose that we remove those usages by:
> 
> jvmci.hpp: Use a forward declaration
> collectedHeap.hpp: Move its usage to the cpp file
> 
> events.hpp also includes thread.hpp, which brings in a large portion of the HotSpot headers. So, this change is also good in that regard.

This pull request has now been integrated.

Changeset: 3462f7a9
Author:    Stefan Karlsson <stefank at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/3462f7a9
Stats:     58 lines in 7 files changed: 31 ins; 27 del; 0 mod

8256955: Move includes of events.hpp out of header files

Reviewed-by: kbarrett, coleenp

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

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


More information about the hotspot-dev mailing list