RFR(XS): 8138896: C1: NativeGeneralJump is mixed up with NativeCall in C1 patching code

Vladimir Ivanov vladimir.x.ivanov at oracle.com
Wed Nov 25 11:24:59 UTC 2015

Looks good to me as well.

Will push it later this week.

Best regards,
Vladimir Ivanov

On 11/24/15 10:04 PM, Christian Thalinger wrote:
>> On Nov 23, 2015, at 12:44 AM, Doerr, Martin <martin.doerr at sap.com
>> <mailto:martin.doerr at sap.com>> wrote:
>> Hi,
>> I have a webrev to clean up the usage of NativeGeneralJump where it is
>> required. Seems like on other platforms than PPC64 NativeGeneralJump
>> and NativeCall have the same size, so it is working by chance.
>> This change is a prerequisite for our C1 on PPC64 contribution where
>> we observed crashes due to this bug.
>> The webrev is here:
>> http://cr.openjdk.java.net/~mdoerr/8138896_c1_patching_fix/webrev.00/
>> @Christian:
>> You already had a look at it. Can I consider it as reviewed?
> Yes, reviewed.
>> I will also need a sponsor, please.
>> Best regards,
>>   Martin

More information about the hotspot-compiler-dev mailing list