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

Introduce umbrella header os.inline.hpp and clean up includes.

XMLWordPrintable

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


        This change adds a new header os.inline.hpp including the os_<os>.include.hpp headers. This allows to remove around 30 os dependent include cascades, some of them without adding the os.inline.hpp header. Also, os.inline.hpp is added in many files that call functions from these headers where it was missing so far.

        Some further cleanups:

        OrderAccess include in adaptiveFreeList.cpp is needed because of freeChunk.hpp.

        The include of os.inline.hpp in thread.inline.hpp is needed because
        Thread::current() uses thread() from ThreadLocalStorage, which again uses
        os::thread_local_storage_at which is implemented platform dependent.

        I moved some methods without dependencies to other .inlclud.hpp files
        to os_windows.hpp/os_posix.hpp. This reduces the need for os.inline.hpp
        includes a lot.

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

                Created:
                Updated:
                Resolved: