RFR: 8257565: epsilonBarrierSet.hpp should not include barrierSetAssembler [v3]

Ioi Lam iklam at openjdk.java.net
Wed Dec 2 07:58:07 UTC 2020


> 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%.

Ioi Lam has updated the pull request incrementally with one additional commit since the last revision:

  fixed aarch64 build

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

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/1554/files
  - new: https://git.openjdk.java.net/jdk/pull/1554/files/82b91085..1df60549

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=1554&range=02
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=1554&range=01-02

  Stats: 1 line in 1 file changed: 1 ins; 0 del; 0 mod
  Patch: https://git.openjdk.java.net/jdk/pull/1554.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/1554/head:pull/1554

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


More information about the hotspot-dev mailing list