[aarch64-port-dev ] RFR: 8217368: AArch64: C2 recursive stack locking optimisation not triggered

[Andrew Haley]

On 1/18/19 5:29 PM, Derek White wrote:

> The original code used cmpxchgptr, but it introduced too many
> unnecessary branches. So you

Me, I think.

> or Ed changed it to this code, with a (7-8 line) comment "Formerly:
> __ cmpxchgptr" etc, etc. I thought that comment didn't add much for
> all that bulk so I asked Nick to rip the comment out!
> 
> The function now fits on one screen (of sufficient size) though.
> 
> Getting cmpxchgptr to work without the extra branches would be a
> better solution if someone has any thoughts in that direction.

There aren't any extra branches if you use MacroAssembler::cmpxchg.

-- 
Andrew Haley
Java Platform Lead Engineer
Red Hat UK Ltd. <https://www.redhat.com>
EAC8 43EB D3EF DB98 CC77 2FAD A5CD 6035 332F A671