[aarch64-port-dev ] RFR: 8216350: AArch64: monitor unlock fast path not called
Nick Gasson (Arm Technology China)
Nick.Gasson at arm.com
Fri Jan 11 02:36:47 UTC 2019
Hi all,
On 09/01/2019 17:23, Andrew Haley wrote:
>
> HotSpot policy is that we can do minor cleanups as we go along:
> experience has shown that unless you do so, cruft tends to
> accumulate. These cleanups are OK for this patch.
>
Please see the updated webrev here:
http://cr.openjdk.java.net/~ngasson/8216350/webrev.1/
Includes cleanups according to Derek's comments and updated the
copyright year (thanks Felix).
> 4) Slightly better comment for last instruction of fast_unlock (and explicitly use zr).
> __ stlr(zr, tmp); // set unowned
Note I needed to change the definition of load_store_exclusive to allow
ZR here. I've checked that this is OK for the other instructions that
use this.
Thanks,
Nick
More information about the hotspot-compiler-dev
mailing list