JDK-8223911 : Disable bad node budget verification until the fix
  • Type: Bug
  • Component: hotspot
  • Sub-Component: compiler
  • Affected Version: 13
  • Priority: P2
  • Status: Closed
  • Resolution: Fixed
  • Submitted: 2019-05-14
  • Updated: 2019-08-15
  • Resolved: 2019-05-15
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 13
13 b21Fixed
Related Reports
Relates :  
JDK-8216137 - assert(Compile::current()->live_nodes() < Compile::current()->max_node_limit()) failed: Live Node limit exceeded limit
Relates :  
JDK-8223363 - Bad node estimate assertion failure
Relates :  
JDK-8223502 - Node estimate for loop unswitching is not correct: assert(delta <= 2 * required) failed: Bad node estimate
Description
Fix for JDK-8216137 regressed fastdebug builds on larger applications with spurious asserts. JDK-8223502 and JDK-8223363 are evidences in the wild. While Patric is looking how to fix those, I suggest we disable the verification to unbreak the fastdebug.
Comments
ILW = Same as JDK-8223363 = P2
15-05-2019
RFR/RFC: https://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2019-May/033729.html
14-05-2019
8223911: Disable bad node budget verification until the fix Reviewed-by: XXX diff -r 809cbe8565a1 -r ee3c58aebd44 src/hotspot/share/opto/loopnode.hpp --- a/src/hotspot/share/opto/loopnode.hpp Tue May 14 20:05:43 2019 +0200 +++ b/src/hotspot/share/opto/loopnode.hpp Tue May 14 20:06:51 2019 +0200 @@ -1384,7 +1384,8 @@ uint required = _nodes_required; require_nodes_final(); uint delta = C->live_nodes() - live_at_begin; - assert(delta <= 2 * required, "Bad node estimate (actual: %d, request: %d)", + // Assert is disabled, see JDK-8223911 and related issues. + assert(true || delta <= 2 * required, "Bad node estimate (actual: %d, request: %d)", delta, required); }
14-05-2019