review (S) for 6927049: assert(is_Loop(),"invalid node class")
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.
> Tom Rodriguez wrote:
>> 6927049: assert(is_Loop(),"invalid node class")
>> 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.
More information about the hotspot-compiler-dev