Clean up synchronizer.hpp

XMLWordPrintable

    • Type: Enhancement
    • Resolution: Fixed
    • Priority: P4
    • 20
    • Affects Version/s: 20
    • Component/s: hotspot
    • b12

      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's included 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 change (out of ~1000 hotspot .o files)

      linkedlist.hpp 857 -> 101
      synchronizer.hpp 848 -> 122
      resourceHash.hpp 850 -> 270

            Assignee:
            Ioi Lam
            Reporter:
            Ioi Lam
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved: