RFR: 8257731: Remove excessive include of stubRoutines.hpp
Ioi Lam
iklam at openjdk.java.net
Thu Dec 3 23:32:10 UTC 2020
stubRoutines.hpp is unnecessarily included by thread.hpp and copy.hpp. This causes a large number of header files related to native code generation to be included by almost all HotSpot .o files.
- Before the fix, stubRoutines.hpp is included by 803 .o files.
- After the fix, stubRoutines.hpp is included by 117 .o files.
The total number of included header files decreased from 252893 to 247631, or about 2%.
Note: the main change is the removal of stubRoutines.hpp in thread.hpp and copy.hpp. Unfortunately I have to fix quite a few missing dependencies in other source files that are revealed by the removal.
-------------
Commit messages:
- 8257731: Remove excessive include of stubRoutines.hpp
Changes: https://git.openjdk.java.net/jdk/pull/1610/files
Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=1610&range=00
Issue: https://bugs.openjdk.java.net/browse/JDK-8257731
Stats: 99 lines in 59 files changed: 73 ins; 10 del; 16 mod
Patch: https://git.openjdk.java.net/jdk/pull/1610.diff
Fetch: git fetch https://git.openjdk.java.net/jdk pull/1610/head:pull/1610
PR: https://git.openjdk.java.net/jdk/pull/1610
More information about the hotspot-dev
mailing list