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