RFR(S): 8210392: assert(Compile::current()->live_nodes() < Compile::current()->max_node_limit()) failed: Live Node limit exceeded limit

[Tobias Hartmann]

Hi Patric,

were you able to reproduce this with a test (I see that one is attached to the bug)? If so, please
add it to the webrev. Please also remove the extra newlines (for example, in line 1146).

The comment in line 1027 says "Use same limit as split_if_with_blocks_post". I think this is
outdated right?

Best regards,
Tobias

On 18.12.18 12:48, Patric Hedlin wrote:
> Dear all,
> 
> I would like to ask for help to review the following change/update:
> 
> Issue:  https://bugs.openjdk.java.net/browse/JDK-8210392
> 
> Webrev: http://cr.openjdk.java.net/~phedlin/tr8210392/
> 
> 
> 8210392: assert(Compile::current()->live_nodes() < Compile::current()->max_node_limit()) failed:
> Live Node limit exceeded limit
> 
>     Avoid excessive split-if through a crude throttling approach.
> 
> 
> Testing: hs-tier1-4, hs-precheckin-comp
> 
> 
> Best regards,
> Patric