--- old/src/hotspot/share/gc/z/zBarrierSet.inline.hpp 2019-11-21 11:55:16.021051360 +0100 +++ new/src/hotspot/share/gc/z/zBarrierSet.inline.hpp 2019-11-21 11:55:15.761046965 +0100 @@ -132,16 +132,16 @@ template template -inline oop ZBarrierSet::AccessBarrier::oop_atomic_cmpxchg_in_heap(oop new_value, T* addr, oop compare_value) { +inline oop ZBarrierSet::AccessBarrier::oop_atomic_cmpxchg_in_heap(T* addr, oop compare_value, oop new_value) { verify_decorators_present(); verify_decorators_absent(); ZBarrier::load_barrier_on_oop_field(addr); - return Raw::oop_atomic_cmpxchg_in_heap(new_value, addr, compare_value); + return Raw::oop_atomic_cmpxchg_in_heap(addr, compare_value, new_value); } template -inline oop ZBarrierSet::AccessBarrier::oop_atomic_cmpxchg_in_heap_at(oop new_value, oop base, ptrdiff_t offset, oop compare_value) { +inline oop ZBarrierSet::AccessBarrier::oop_atomic_cmpxchg_in_heap_at(oop base, ptrdiff_t offset, oop compare_value, oop new_value) { verify_decorators_present(); verify_decorators_absent(); @@ -150,7 +150,7 @@ // with the motivation that if you're doing Unsafe operations on a Reference.referent // field, then you're on your own anyway. ZBarrier::load_barrier_on_oop_field(field_addr(base, offset)); - return Raw::oop_atomic_cmpxchg_in_heap_at(new_value, base, offset, compare_value); + return Raw::oop_atomic_cmpxchg_in_heap_at(base, offset, compare_value, new_value); } template @@ -222,11 +222,11 @@ template template -inline oop ZBarrierSet::AccessBarrier::oop_atomic_cmpxchg_not_in_heap(oop new_value, T* addr, oop compare_value) { +inline oop ZBarrierSet::AccessBarrier::oop_atomic_cmpxchg_not_in_heap(T* addr, oop compare_value, oop new_value) { verify_decorators_present(); verify_decorators_absent(); - return Raw::oop_atomic_cmpxchg_not_in_heap(new_value, addr, compare_value); + return Raw::oop_atomic_cmpxchg_not_in_heap(addr, compare_value, new_value); } template