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

[Vladimir Ivanov]

Looks good.

Best regards,
Vladimir Ivanov

On 30/04/2019 07:28, 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-8216137
> Webrev: http://cr.openjdk.java.net/~phedlin/tr8216137/
> 
> 8216137: assert(Compile::current()->live_nodes() < 
> Compile::current()->max_node_limit()) failed:
>           Live Node limit exceeded limit
> 
> Also addressed:
> 
> 8219520: assert(Compile::current()->live_nodes() < 
> Compile::current()->max_node_limit()) failed:
>           Live Node limit exceeded limit
> 
> Approach:
> 
>      Adding a simplistic (ad-hoc) node budget mechanism, applied during 
> loop transforms.
> 
> 
> Testing: hs-tier1..4, hs-precheckin-comp, Kitchensink24h
> 
> 
> Caveat:  Testing and benchmarking needs to be reran but is currently 
> experiencing issues.
> 
> 
> Best regards,
> Patric
>