[aarch64-port-dev ] RFR: 8217368: AArch64: C2 recursive stack locking optimisation not triggered
Nick Gasson (Arm Technology China)
Nick.Gasson at arm.com
Fri Jan 18 09:52:50 UTC 2019
On 18/01/2019 17:36, Andrew Haley wrote:
> The patch looks good. However, I don't understand why we aren't using
> MacroAssembler::cmpxchgptr here. It looks like we should be, and you'd
> end up with a less complex result.
It's not exactly the same though: MacroAssembler::cmpxchgptr adds a "dmb
ish" to the failure path which I don't think is required here.
>> * Does anyone know what the comment "// Load Compare Value application
>> register." means? It's present in the PPC and S390 ports too.
> Probably no-one can remember. We'll have inherited it from x86.
Let's delete it then.
More information about the hotspot-compiler-dev