review (XS) for 7026957: assert(type2aelembytes(store->as_Mem()->memory_type(), true) == 1 << shift->in(2)->get_int()) failed

Tom Rodriguez tom.rodriguez at oracle.com
Thu Mar 31 21:18:02 PDT 2011


On Mar 31, 2011, at 9:08 PM, John Rose wrote:

> Reviewed.  (Consider parenthesizing the "<<" expression.)  -- John

Done.  Thanks John and Vladimir.

tom

> 
> On Mar 31, 2011, at 8:57 PM, Tom Rodriguez wrote:
> 
>> http://cr.openjdk.java.net/~never/7026957
>> 
>> 7026957: assert(type2aelembytes(store->as_Mem()->memory_type(), true) == 1 << shift->in(2)->get_int()) failed
>> Reviewed-by:
>> 
>> The assert that checks that the shift matches the store size should
>> really be a test and bailout if they don't match.  Tested with failing
>> test from report.
>> 
> 



More information about the hotspot-compiler-dev mailing list