RFR: 8360654: AArch64: Remove redundant dmb from C1 compareAndSet [v5]

[Andrew Haley]

On Wed, 12 Nov 2025 14:16:41 GMT, Andrew Haley <aph at openjdk.org> wrote:

>> Samuel Chee has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains six additional commits since the last revision:
>> 
>>  - Address review comments. Refine.
>>  - Merge from the main branch
>>  - Add cmpxchg_barrier helper
>>    
>>    Change-Id: I17acf999140f0c1decb256de8291361c568a4ff8
>>  - Add comment
>>    
>>    Signed-off-by: Samuel Chee <samche01 at arm.com>
>>    Change-Id: I9793ed6ffdff6c044552d069af23620d178f2284
>>  - Add back in dmb membar for non-LSE
>>    
>>    Change-Id: Ie64565420a1758d3191eaebed82c80584ce54ef6
>>  - 8360654: AArch64: Remove redundant dmb from C1 compareAndSet
>>    
>>    Change-Id: I79a0079fc2d3d90eeb671b6ed73d963968d4fa53
>
> src/hotspot/cpu/aarch64/macroAssembler_aarch64.cpp line 3465:
> 
>> 3463: }
>> 3464: 
>> 3465: void MacroAssembler::cmpxchg(Register addr, Register expected,
> 
> Why do we need all of these non-barrier versions?

Ping? I don't know what this is for. C1 will only add barriers for pre-LSE systems, and nothing else cares. You've got several new methods that no one needs.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/26000#discussion_r2541963608