JDK-8236178 : Debug build failed after 8236058
  • Type: Bug
  • Component: hotspot
  • Sub-Component: compiler
  • Affected Version: openjdk8u242
  • Priority: P2
  • Status: Resolved
  • Resolution: Fixed
  • Submitted: 2019-12-18
  • Updated: 2020-02-17
  • Resolved: 2019-12-25
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.
Other
openjdk8u242 b05Fixed
Related Reports
Relates :  
Relates :  
Description
/jdk8u/hotspot/src/share/vm/oops/instanceKlass.cpp: In member function ���void InstanceKlass::set_init_state(InstanceKlass::ClassState)���:
/jdk8u/hotspot/src/share/vm/oops/instanceKlass.cpp:3608:111: error: ���set_initialization_state_and_notify_implassert��� was not declared in this scope
   set_initialization_state_and_notify_implassert(_init_thread == NULL, "should be cleared before state change");
Comments
URL: https://hg.openjdk.java.net/jdk8u/jdk8u/hotspot/rev/2c1e9fab6964 User: andrew Date: 2019-12-25 03:01:44 +0000
25-12-2019

RFR: http://mail.openjdk.java.net/pipermail/jdk8u-dev/2019-December/010822.html I'll let Andrew do the approval since he's currently working on -b05. +1 on jdk8u-critical-yes
18-12-2019

Suggested fix diff -r 9ef81b9152f1 src/share/vm/oops/instanceKlass.cpp --- a/src/share/vm/oops/instanceKlass.cpp Fri Nov 13 18:14:41 2015 +0300 +++ b/src/share/vm/oops/instanceKlass.cpp Wed Dec 18 14:25:28 2019 +0300 @@ -3605,7 +3605,7 @@ bool good_state = is_shared() ? (_init_state <= state) : (_init_state < state); assert(good_state || state == allocated, "illegal state transition"); - set_initialization_state_and_notify_implassert(_init_thread == NULL, "should be cleared before state change"); + assert(_init_thread == NULL, "should be cleared before state change"); _init_state = (u1)state; } #endif
18-12-2019

may I please suggest to include this trivial fix into 8u242 in order to have buildable codebase
18-12-2019