Integrated: 8264748: Do not include arguments.hpp from compilerDefinitions.hpp

Ioi Lam iklam at openjdk.java.net
Wed Apr 7 05:41:26 UTC 2021


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

> compilerDefinitions.hpp is a popular header, included by 588 of about 1000 HotSpot .o files. It includes arguments.hpp only for a single function that's not used in critical paths.
> 
> After refactoring, the number of .o files that includes arguments.hpp reduces from 623 to 68.
> 
> (I also removed arguments.hpp from other files that don't actually need it.)

This pull request has now been integrated.

Changeset: 17202c89
Author:    Ioi Lam <iklam at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/17202c89
Stats:     59 lines in 41 files changed: 16 ins; 33 del; 10 mod

8264748: Do not include arguments.hpp from compilerDefinitions.hpp

Reviewed-by: gziemski, stefank

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

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


More information about the hotspot-dev mailing list