RFR(XXS) 8027751: C1 crashes in Weblogic with G1 enabled

[Igor Veresov]

Sure. IgorI also insists I add a regression test.

The updated webrev: http://cr.openjdk.java.net/~iveresov/8027751/webrev.1/

igor 


On Nov 4, 2013, at 8:52 AM, Vladimir Kozlov <vladimir.kozlov at oracle.com> wrote:

> Can you move && inside macro to avoid NOT_LP64()?:
> 
> LP64_ONLY(&& (opr_type != T_OBJECT))
> 
> Thanks,
> Vladimir
> 
> On 11/4/13 12:59 AM, Igor Veresov wrote:
>> G1 barriers use logical operators (xor) on T_OBJECT mixed with T_LONG or T_INT. The current implementation of logical operations on x86 in C1 doesn't allow for long operands to be on stack. There is a special code in the register allocator that forces long arguments in registers on x86. However T_OBJECT can be spilled just fine, and in that case the xor emission will fail.
>> 
>> The shortest fix is to force the allocator to keep operands of type T_OBJECT in registers when doing logical operations (as we do with T_LONG already) on 64bit x86.
>> 
>> Webrev: http://cr.openjdk.java.net/~iveresov/8027751/webrev.0/
>> 
>> Thanks!
>> igor
>>