JDK-8132286 : change 'InlineNotify' flag option from "product" to "diagnostic"
  • Type: Bug
  • Component: hotspot
  • Sub-Component: runtime
  • Affected Version: 9
  • Priority: P3
  • Status: Resolved
  • Resolution: Fixed
  • OS: generic
  • CPU: generic
  • Submitted: 2015-07-23
  • Updated: 2015-09-10
  • Resolved: 2015-07-24
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 9
9 b77Fixed
Related Reports
Relates :  
Description
The 'InlineNotify' flag option that was recently added by JDK-8075171
as a "product" flag should have been added as a "diagnostic" flag.
Diagnostic flags are easier to retire.
Comments
I am happy it isn't a product flag, but the change to diagnostic still requires a CCC AFAIU. I'll ask Joe Darcy.
25-07-2015

$ hg diff diff -r 5ec83d7b8a2d src/share/vm/runtime/globals.hpp --- a/src/share/vm/runtime/globals.hpp Thu Jul 23 16:36:36 2015 +0000 +++ b/src/share/vm/runtime/globals.hpp Thu Jul 23 14:32:24 2015 -0700 @@ -1286,7 +1286,7 @@ public: \ experimental(intx, SyncVerbose, 0, "(Unstable)") \ \ - product(bool, InlineNotify, true, "intrinsify subset of notify" ) \ + diagnostic(bool, InlineNotify, true, "intrinsify subset of notify") \ \ experimental(intx, ClearFPUAtPark, 0, "(Unsafe, Unstable)") \ \ Also got rid of the space before the ')'.
23-07-2015