RFR: 8264797: Do not include klassVtable.hpp from instanceKlass.hpp

Coleen Phillimore coleenp at openjdk.java.net
Tue Apr 6 19:26:31 UTC 2021


On Tue, 6 Apr 2021 17:07:23 GMT, Ioi Lam <iklam at openjdk.org> wrote:

> instanceKlass.hpp is a popular header, included by 647 of about 1000 HotSpot .o files. It unnecessarily includes klassVtable.hpp (only forward class declarations are needed).
> 
> After refactoring, the number of .o files that includes klassVtable.hpp reduces from 647 to 132.

I thought you'd need to include the file because of this, because it's not a pointer:
  klassItable itable() const;        // return klassItable wrapper

That's good that you don't!

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

Marked as reviewed by coleenp (Reviewer).

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


More information about the hotspot-runtime-dev mailing list