Request for reviews (S): 6860599: Relax nodes limit check for Output phase

Christian Thalinger Christian.Thalinger at Sun.COM
Sun Jul 19 02:53:31 PDT 2009


Vladimir Kozlov wrote:
> http://cr.openjdk.java.net/~kvn/6860599/webrev.00
> 
> Fixed 6860599: Relax nodes limit check for Output phase
> 
> Problem:
> I got several CTW cases when without EA C2 "gracefully"
> bailout compilation when nodes limit check failed during
> macro nodes expansion. And with EA it passed macro nodes
> expansion but crashed with ASSERT during Output phase.
> One byte MachNop nodes are used in debug mode for loops
> and calls alignment in Output phase. As result for a big
> method the node limit could be reached.
> 
> Solution:
> Increase nodes limit (double) for Output phase.

What I don't understand with this patch is, it changes the node limit
but this is done for every output.  It's not limited to e.g. debug mode.
 Is this what you indented?

-- Christian



More information about the hotspot-compiler-dev mailing list