RFR: 8078521: AARCH64: Add AArch64 SA support

Dmitry Samersoff dmitry.samersoff at oracle.com
Wed Jun 24 09:17:31 UTC 2015


Andrew,

I'm ready to push your changes.

Do you have an official jdk9 reviewer sign off?

-Dmitry

On 2015-06-12 17:58, Andrew Haley wrote:
> On 06/12/2015 02:12 PM, Dmitry Samersoff wrote:
>> Andrew,
>>
>> Found minor nit in the fix, please update the webrev:
>>
>> diff -r 640718253035 -r f043d189d1e1
>> agent/src/share/classes/sun/jvm/hotspot/runtime/aarch64/AARCH64Frame.java
>> ---
>> a/agent/src/share/classes/sun/jvm/hotspot/runtime/aarch64/AARCH64Frame.java
>> Tue May 19 15:37:29 2015 +0100
>> +++
>> b/agent/src/share/classes/sun/jvm/hotspot/runtime/aarch64/AARCH64Frame.java
>> Thu Jun 11 16:54:41 2015 +0300
>> @@ -391,7 +391,7 @@
>>        map.setIncludeArgumentOops(cb.callerMustGCArguments());
>>
>>        if (cb.getOopMaps() != null) {
>> -        OopMapSet.updateRegisterMap(this, cb, map, true);
>> +        ImmutableOopMapSet.updateRegisterMap(this, cb, map, true);
>>        }
>>
>>        // Since the prolog does the save and restore of FP there is no
>> oopmap
> 
> 
> http://cr.openjdk.java.net/~aph/8078521-6/
> 
> Andrew.
> 


-- 
Dmitry Samersoff
Oracle Java development team, Saint Petersburg, Russia
* I would love to change the world, but they won't give me the sources.


More information about the serviceability-dev mailing list