Integrated: 8292267: Clean up synchronizer.hpp

[Ioi Lam]

On Fri, 12 Aug 2022 00:42:52 GMT, Ioi Lam <iklam at openjdk.org> wrote:

> 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

This pull request has now been integrated.

Changeset: 7244dd6f
Author:    Ioi Lam <iklam at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/7244dd6fab0c516ed76af594593b8378512620c8
Stats:     54 lines in 21 files changed: 28 ins; 21 del; 5 mod

8292267: Clean up synchronizer.hpp

Reviewed-by: coleenp, dcubed, dholmes

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

PR: https://git.openjdk.org/jdk/pull/9849