RFR(XXS): A couple of type fixes in C1

Igor Veresov igor.veresov at oracle.com
Thu Mar 13 21:53:24 UTC 2014


Thanks, Chris!

igor

On Mar 13, 2014, at 1:43 PM, Christian Thalinger <christian.thalinger at oracle.com> wrote:

> Looks good.
> 
> On Mar 13, 2014, at 10:20 AM, Igor Veresov <igor.veresov at oracle.com> wrote:
> 
>> I’d like to push a couple of tiny fixes of types in C1 that went unnoticed when porting to 64-bit.
>> 
>> 8037140: C1: Incorrect argument type used for SharedRuntime::OSR_migration_end in LIRGenerator::do_Goto
>> 8037149: C1: getThreadTemp should return a T_LONG register on 64bit
>> 
>> The fixes don’t really change the generated code since the only affect some reg-reg moves. However it’d be nice to improve the overall type hygiene.
>> 
>> Webrevs:
>> http://cr.openjdk.java.net/~iveresov/8037149/webrev.00/
>> http://cr.openjdk.java.net/~iveresov/8037140/webrev.00/
>> 
>> Thanks,
>> igor
> 



More information about the hotspot-compiler-dev mailing list