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

Tom Rodriguez Thomas.Rodriguez at Sun.COM
Fri Feb 19 12:19:25 PST 2010

Sorry, I should have claimed it.  Tony helped me reproduce it the day before yesterday but I just got around to sending out the review.  Thanks!


On Feb 19, 2010, at 12:06 PM, Vladimir Kozlov wrote:

> Thank you, Tom
> You are fast, I just tried to find this bug report few minutes ago :)
> Looks good.
> Thanks,
> Vladimir
> 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