Integrated: 8257565: epsilonBarrierSet.hpp should not include barrierSetAssembler
Ioi Lam
iklam at openjdk.java.net
Wed Dec 2 16:59:57 UTC 2020
On Wed, 2 Dec 2020 05:22:04 GMT, Ioi Lam <iklam at openjdk.org> wrote:
> Please review this trivial fix:
>
> epsilonBarrierSet.hpp is included (recursively via access.hpp) by many CPP files. It unncessarily includes of barrierSetAssembler.hpp, which causes many of the native code assembler header files to be unnecessarily included by many HotSpot CPP files that do not deal with native code assembly.
>
> Removing this one line reduced the total number of header inclusion for building HotSpot from 260096 to 258193, or about 0.8%.
This pull request has now been integrated.
Changeset: 67042664
Author: Ioi Lam <iklam at openjdk.org>
URL: https://git.openjdk.java.net/jdk/commit/67042664
Stats: 3 lines in 3 files changed: 2 ins; 1 del; 0 mod
8257565: epsilonBarrierSet.hpp should not include barrierSetAssembler
Reviewed-by: kbarrett, stuefe, shade
-------------
PR: https://git.openjdk.java.net/jdk/pull/1554
More information about the hotspot-dev
mailing list