Request for reviews (S): 6860599: Relax nodes limit check for Output phase
Christian.Thalinger at Sun.COM
Sun Jul 19 02:53:31 PDT 2009
Vladimir Kozlov wrote:
> Fixed 6860599: Relax nodes limit check for Output phase
> 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.
> 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?
More information about the hotspot-compiler-dev