Integrated: 8310076: Reduce inclusion of bytecodeStream.hpp

Ioi Lam iklam at openjdk.org
Fri Jun 16 15:29:10 UTC 2023


On Thu, 15 Jun 2023 01:17:26 GMT, Ioi Lam <iklam at openjdk.org> wrote:

> The popular header file generateOopMap.hpp includes bytecodeStream.hpp, but it doesn't actually needs the full declaration of the BytecodeStream class.
> 
> This inclusion can be replaced with a simple forward declaration of the BytecodeStream class.
> 
> This reduces the number of HotSpot .o files that include bytecodeStream.hpp from 571 to 21.

This pull request has now been integrated.

Changeset: a45f9296
Author:    Ioi Lam <iklam at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/a45f92961bd0dd603daad8149da4a976d60425f9
Stats:     13 lines in 7 files changed: 9 ins; 1 del; 3 mod

8310076: Reduce inclusion of bytecodeStream.hpp

Reviewed-by: dholmes, coleenp

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

PR: https://git.openjdk.org/jdk/pull/14481


More information about the hotspot-dev mailing list