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