[prev in list] [next in list] [prev in thread] [next in thread]
List: openjdk-serviceability-dev
Subject: Re: PING Re: RFR: 8078521: AARCH64: Add AArch64 SA support
From: Andrew Haley <aph () redhat ! com>
Date: 2015-06-12 14:58:42
Message-ID: 557AF3A2.4010508 () redhat ! com
[Download RAW message or body]
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.
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic