RFR: 8292267: Clean up synchronizer.hpp
Ioi Lam
iklam at openjdk.org
Fri Aug 12 03:15:02 UTC 2022
Fix two problems with synchronizer.hpp:
- It's unnecessarily included in two popular headers: monitorChunk.hpp and frame_<cpu\>.hpp. The latter is most problematic because it includes synchronizer.hpp in the middle of a class declaration!
- synchronizer.hpp includes linkedlist.hpp. This can be avoid by moving the definition of Synchronizer::PtrList into synchronizer.cpp.
These headers are included much less after this PR (out of ~1000 hotspot .o files)
linkedlist.hpp 857 -> 101
synchronizer.hpp 848 -> 122
resourceHash.hpp 850 -> 270
-------------
Commit messages:
- added back inlined functions in case their performance matters
- fixed copyright
- 8292267: Clean up synchronizer.hpp
Changes: https://git.openjdk.org/jdk/pull/9849/files
Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=9849&range=00
Issue: https://bugs.openjdk.org/browse/JDK-8292267
Stats: 54 lines in 21 files changed: 27 ins; 20 del; 7 mod
Patch: https://git.openjdk.org/jdk/pull/9849.diff
Fetch: git fetch https://git.openjdk.org/jdk pull/9849/head:pull/9849
PR: https://git.openjdk.org/jdk/pull/9849
More information about the shenandoah-dev
mailing list