Uploaded image for project: 'JDK'
  1. JDK
  2. JDK-8292267

Clean up synchronizer.hpp

XMLWordPrintable

    • Icon: Enhancement Enhancement
    • Resolution: Fixed
    • Icon: P4 P4
    • 20
    • 20
    • 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

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

              Created:
              Updated:
              Resolved: