--- old/src/hotspot/share/oops/oop.inline.hpp 2017-10-10 17:34:16.152698778 -0400 +++ new/src/hotspot/share/oops/oop.inline.hpp 2017-10-10 17:34:15.694954540 -0400 @@ -66,7 +66,7 @@ template void oop_store(volatile T* p, oop v) { update_barrier_set_pre((T*)p, v); // cast away volatile - // Used by release_obj_field_put, so use release_store_ptr. + // Used by release_obj_field_put, so use release_store. oopDesc::release_encode_store_heap_oop(p, v); // When using CMS we must mark the card corresponding to p as dirty // with release sematics to prevent that CMS sees the dirty card but @@ -90,7 +90,7 @@ // We need a separate file to avoid circular references void oopDesc::release_set_mark(markOop m) { - OrderAccess::release_store_ptr(&_mark, m); + OrderAccess::release_store(&_mark, m); } markOop oopDesc::cas_set_mark(markOop new_mark, markOop old_mark) { @@ -124,7 +124,7 @@ volatile narrowKlass* xaddr = const_cast(addr); return Klass::decode_klass(OrderAccess::load_acquire(xaddr)); } else { - return (Klass*)OrderAccess::load_ptr_acquire(&_metadata._klass); + return OrderAccess::load_acquire(&_metadata._klass); } } @@ -161,7 +161,7 @@ OrderAccess::release_store(compressed_klass_addr(), Klass::encode_klass_not_null(k)); } else { - OrderAccess::release_store_ptr(klass_addr(), k); + OrderAccess::release_store(klass_addr(), k); } } @@ -361,7 +361,7 @@ // Store heap oop as is for volatile fields. void oopDesc::release_store_heap_oop(volatile oop* p, oop v) { - OrderAccess::release_store_ptr(p, v); + OrderAccess::release_store(p, v); } void oopDesc::release_store_heap_oop(volatile narrowOop* p, narrowOop v) { OrderAccess::release_store(p, v); @@ -372,11 +372,11 @@ OrderAccess::release_store(p, encode_heap_oop_not_null(v)); } void oopDesc::release_encode_store_heap_oop_not_null(volatile oop* p, oop v) { - OrderAccess::release_store_ptr(p, v); + OrderAccess::release_store(p, v); } void oopDesc::release_encode_store_heap_oop(volatile oop* p, oop v) { - OrderAccess::release_store_ptr(p, v); + OrderAccess::release_store(p, v); } void oopDesc::release_encode_store_heap_oop(volatile narrowOop* p, oop v) { OrderAccess::release_store(p, encode_heap_oop(v)); @@ -392,7 +392,7 @@ // decode old from T to oop return decode_heap_oop(old); } else { - return (oop)Atomic::xchg_ptr(exchange_value, (oop*)dest); + return Atomic::xchg(exchange_value, (oop*)dest); } } @@ -447,11 +447,11 @@ void oopDesc::metadata_field_put(int offset, Metadata* value) { *metadata_field_addr(offset) = value; } Metadata* oopDesc::metadata_field_acquire(int offset) const { - return (Metadata*)OrderAccess::load_ptr_acquire(metadata_field_addr(offset)); + return OrderAccess::load_acquire(metadata_field_addr(offset)); } void oopDesc::release_metadata_field_put(int offset, Metadata* value) { - OrderAccess::release_store_ptr(metadata_field_addr(offset), value); + OrderAccess::release_store(metadata_field_addr(offset), value); } jbyte oopDesc::byte_field(int offset) const { return (jbyte) *byte_field_addr(offset); } @@ -485,8 +485,8 @@ return UseCompressedOops ? decode_heap_oop((narrowOop) OrderAccess::load_acquire(obj_field_addr(offset))) - : decode_heap_oop((oop) - OrderAccess::load_ptr_acquire(obj_field_addr(offset))); + : decode_heap_oop( + OrderAccess::load_acquire(obj_field_addr(offset))); } void oopDesc::release_obj_field_put(int offset, oop value) { UseCompressedOops ? @@ -518,8 +518,8 @@ jdouble oopDesc::double_field_acquire(int offset) const { return OrderAccess::load_acquire(double_field_addr(offset)); } void oopDesc::release_double_field_put(int offset, jdouble contents) { OrderAccess::release_store(double_field_addr(offset), contents); } -address oopDesc::address_field_acquire(int offset) const { return (address) OrderAccess::load_ptr_acquire(address_field_addr(offset)); } -void oopDesc::release_address_field_put(int offset, address contents) { OrderAccess::release_store_ptr(address_field_addr(offset), contents); } +address oopDesc::address_field_acquire(int offset) const { return OrderAccess::load_acquire(address_field_addr(offset)); } +void oopDesc::release_address_field_put(int offset, address contents) { OrderAccess::release_store(address_field_addr(offset), contents); } bool oopDesc::is_locked() const { return mark()->is_locked();