review (XS) for 6921992: failure in verify scheduling after 6792161

Vladimir Kozlov Vladimir.Kozlov at Sun.COM
Thu Feb 4 11:13:33 PST 2010


Agree.

Vladimir

Tom Rodriguez wrote:
> Yes we can live without the fix though I think we need to fix it at some point.  I'm going to push this to hotspot/hotspot first since these changes just were pushed up and I'd prefer the failure not to propagate out further.
> 
> tom
> 
> On Feb 4, 2010, at 10:48 AM, Vladimir Kozlov wrote:
> 
>> Changes look good.
>>
>> Tom, was 679216 case rare? Can we live without the fix?
>>
>> Thanks,
>> Vladimir
>>
>> Tom Rodriguez wrote:
>>> http://cr.openjdk.java.net/~never/6921992
>>> 6921992: failure in verify scheduling after 6792161
>>> Reviewed-by:
>>> The fix for 6792161 created problems with the scheduler verification
>>> logic so until I find a better for 6792161 I'm backing it out.  This
>>> is just an anti-delta for the original fix.
>>> src/share/vm/opto/ifg.cpp
>>> The previous changeset was http://hg.openjdk.java.net/jdk7/hotspot-comp/hotspot/rev/0e14bd797dad
> 


More information about the hotspot-compiler-dev mailing list