Integrated: 8259894: refactor parts of jvm.h into jvm_io.h and jvm_constants.h

Ioi Lam iklam at openjdk.java.net
Mon Jan 25 08:03:46 UTC 2021


On Wed, 20 Jan 2021 04:33:56 GMT, Ioi Lam <iklam at openjdk.org> wrote:

> jvm.h is over 1000 lines long. It is included by 940 out of 1000 HotSpot .o files. However, most of the HotSpot source files just use a small fraction of jvm.h:
> 
> - the jio_xxxprintf() functions
> - the JVM_RECOGNIZED_XXX_MODIFIERS defines
> 
> We should move these two parts into jvm_io.h and jvm_constants.h. This reduces the files that include jvm.h to about 82 files. Build time of HotSpot is reduced by about 0.4%.
> 
> Testing: mach5: tier1, builds-tier2, builds-tier3, builds-tier4 and builds-tier5. Also locally: aarch64, arm, ppc64, s390, x86, and zero.

This pull request has now been integrated.

Changeset: 5898ab65
Author:    Ioi Lam <iklam at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/5898ab65
Stats:     252 lines in 17 files changed: 145 ins; 86 del; 21 mod

8259894: refactor parts of jvm.h into jvm_io.h and jvm_constants.h

Reviewed-by: dholmes, coleenp

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

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


More information about the hotspot-dev mailing list