[aarch64-port-dev ] RFR: 8216557 Aarch64: Add support for Concurrent Class Unloading

[Andrew Haley]

On 3/26/20 10:42 PM, Stuart Monteith wrote:
> 
>         BarrierSetAssembler::nmethod_entry_barrier
>         This method emits the barrier code. In internal review it was suggested
> the "dmb( ISHLD )" should be replaced by "membar(LoadLoad)". I've not
> done this as the BarrierSetNMethod code checks the exact instruction
> sequence, and I prefer to be explicit.

I understand, but LoadLoad is the semantics you need, and it's more important
to say that. The mere existence of verification code shouldn't determine
how you express the runtime code.

I'll do a thorough review later.

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