JDK-8194898 : Move OopStorage inline definitions to an .inline.hpp
  • Type: Enhancement
  • Component: hotspot
  • Sub-Component: gc
  • Priority: P4
  • Status: Resolved
  • Resolution: Fixed
  • OS: generic
  • CPU: generic
  • Submitted: 2018-01-10
  • Updated: 2018-02-16
  • Resolved: 2018-01-29
The Version table provides details related to the release that this issue/RFE will be addressed.

Unresolved : Release in which this issue/RFE will be addressed.
Resolved: Release in which this issue/RFE has been resolved.
Fixed : Release in which this issue/RFE has been fixed. The release containing this fix may be available for download as an Early Access Release or a General Availability Release.

To download the current JDK release, click here.
JDK 11
11 b01Fixed
Related Reports
Blocks :  
JDK-8194312 - Support parallel and concurrent JNI global handle processing
Description
gc/shared/oopStorage.hpp contains the class definition for OopStorage, along with a lot of inline function and inner class definitions.  It would be better to move the inline definitions to a .inline.hpp file.  All of those inline definitions are related to iteration, and many clients don't use iteration.  It might even be worthwhile having the following structure:

(1) oopStorage.hpp - contains just class OopStorage.

(2) oopStorage.inline.hpp - contains all the inline definitions
presently in oopStorage.hpp, except for the ParState stuff.

(3) oopStorageParState.inline.hpp - contains the ParState stuff.

Inclusion of the ParState header by non-GC code should be suspect.