review (S) for 6927049: assert(is_Loop(),"invalid node class")

Vladimir Kozlov Vladimir.Kozlov at Sun.COM
Fri Feb 19 12:06:18 PST 2010

Thank you, Tom

You are fast, I just tried to find this bug report few minutes ago :)

Looks good.


Tom Rodriguez wrote:
> 6927049: assert(is_Loop(),"invalid node class")
> Reviewed-by:
> The loop predication code is missing a guard for is_Loop since it's
> possible that the loop might simply by a Region if there are
> irreducible loops in the loop tree.  I also moved the other loop type
> guards up so they would be together before any traps count logic.
> Tested with failing test from report.
> src/share/vm/opto/loopTransform.cpp

More information about the hotspot-compiler-dev mailing list