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

Improve usage of umbrella header atomic.inline.hpp.

XMLWordPrintable

    • Icon: Enhancement Enhancement
    • Resolution: Fixed
    • Icon: P4 P4
    • 9
    • 9
    • hotspot
    • None
    • 9
    • b23

      This change improves usage of umbrella header atomic.inline.hpp.
      It removes includes of this header in files where it's not needed,
      and adds it in all .cpp and .inline.hpp files where a method of
      Atomic declared 'inline' is used.

      Also, the change moves some calls to such methods from .hpp files to
      .inline.hpp files. In case of ASSERT code it moves the calls
      to .cpp files.

      A row of headers still contain calls to inline methods of Atomic,
      which I don't want to move as no appropriate .inline.hpp file is existing:

      src/share/vm/compiler/compileBroker.hpp
      src/share/vm/gc_implementation/concurrentMarkSweep/concurrentMarkSweepThread.hpp
      src/share/vm/gc_implementation/g1/g1StringDedup.hpp
      src/share/vm/gc_implementation/g1/heapRegionRemSet.hpp
      src/share/vm/gc_implementation/parallelScavenge/psParallelCompact.hpp
      src/share/vm/gc_implementation/shared/parGCAllocBuffer.hpp
      src/share/vm/memory/specialized_oop_closures.hpp
      src/share/vm/oops/methodData.hpp
      src/share/vm/runtime/safepoint.hpp
      src/share/vm/services/lowMemoryDetector.hpp
      src/share/vm/services/memTracker.hpp
      src/share/vm/utilities/taskqueue.hpp

            goetz Goetz Lindenmaier
            goetz Goetz Lindenmaier
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved: