Remove inclusion of allocation.hpp from atomicAccess.hpp

XMLWordPrintable

    • Type: Enhancement
    • Resolution: Unresolved
    • Priority: P4
    • 27
    • Affects Version/s: None
    • Component/s: hotspot

      runtime/atomicAccess.hpp includes memory/allocation.hpp. The only things it is using from there are the base classes AllStatic and StackObj. AllStatic can instead be obtained from memory/allStatic.hpp. And StackObj is pretty useless (see JDK-8374333) and definitely not worth pulling in the allocation stuff for one obscure use.

      But the real reason for removing allocation.hpp from the include list for atomicAccess.hpp is that adding includes for runtime/atomic.hpp (which includes atomicAccess.hpp) to certain headers leads to circular include problems.

            Assignee:
            Kim Barrett
            Reporter:
            Kim Barrett
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated: