diff -r 55c43e677ded -r fb0275c320a0 src/hotspot/share/runtime/stubRoutines.cpp --- a/src/hotspot/share/runtime/stubRoutines.cpp Mon Nov 20 12:04:13 2017 +0100 +++ b/src/hotspot/share/runtime/stubRoutines.cpp Mon Nov 20 13:07:44 2017 +0100 @@ -25,6 +25,7 @@ #include "precompiled.hpp" #include "asm/codeBuffer.hpp" #include "memory/resourceArea.hpp" +#include "oops/access.inline.hpp" #include "oops/oop.inline.hpp" #include "runtime/interfaceSupport.hpp" #include "runtime/timerTrace.hpp" @@ -377,19 +378,6 @@ // Default versions of arraycopy functions // -static void gen_arraycopy_barrier_pre(oop* dest, size_t count, bool dest_uninitialized) { - assert(count != 0, "count should be non-zero"); - assert(count <= (size_t)max_intx, "count too large"); - BarrierSet* bs = Universe::heap()->barrier_set(); - bs->write_ref_array_pre(dest, (int)count, dest_uninitialized); -} - -static void gen_arraycopy_barrier(oop* dest, size_t count) { - assert(count != 0, "count should be non-zero"); - BarrierSet* bs = Universe::heap()->barrier_set(); - bs->write_ref_array((HeapWord*)dest, count); -} - JRT_LEAF(void, StubRoutines::jbyte_copy(jbyte* src, jbyte* dest, size_t count)) #ifndef PRODUCT SharedRuntime::_jbyte_array_copy_ctr++; // Slow-path byte array copy @@ -423,9 +411,7 @@ SharedRuntime::_oop_array_copy_ctr++; // Slow-path oop array copy #endif // !PRODUCT assert(count != 0, "count should be non-zero"); - gen_arraycopy_barrier_pre(dest, count, /*dest_uninitialized*/false); - Copy::conjoint_oops_atomic(src, dest, count); - gen_arraycopy_barrier(dest, count); + HeapAccess<>::oop_arraycopy(NULL, NULL, (HeapWord*)src, (HeapWord*)dest, count); JRT_END JRT_LEAF(void, StubRoutines::oop_copy_uninit(oop* src, oop* dest, size_t count)) @@ -433,9 +419,7 @@ SharedRuntime::_oop_array_copy_ctr++; // Slow-path oop array copy #endif // !PRODUCT assert(count != 0, "count should be non-zero"); - gen_arraycopy_barrier_pre(dest, count, /*dest_uninitialized*/true); - Copy::conjoint_oops_atomic(src, dest, count); - gen_arraycopy_barrier(dest, count); + HeapAccess::oop_arraycopy(NULL, NULL, (HeapWord*)src, (HeapWord*)dest, count); JRT_END JRT_LEAF(void, StubRoutines::arrayof_jbyte_copy(HeapWord* src, HeapWord* dest, size_t count)) @@ -471,9 +455,7 @@ SharedRuntime::_oop_array_copy_ctr++; // Slow-path oop array copy #endif // !PRODUCT assert(count != 0, "count should be non-zero"); - gen_arraycopy_barrier_pre((oop *) dest, count, /*dest_uninitialized*/false); - Copy::arrayof_conjoint_oops(src, dest, count); - gen_arraycopy_barrier((oop *) dest, count); + HeapAccess::oop_arraycopy(NULL, NULL, src, dest, count); JRT_END JRT_LEAF(void, StubRoutines::arrayof_oop_copy_uninit(HeapWord* src, HeapWord* dest, size_t count)) @@ -481,9 +463,7 @@ SharedRuntime::_oop_array_copy_ctr++; // Slow-path oop array copy #endif // !PRODUCT assert(count != 0, "count should be non-zero"); - gen_arraycopy_barrier_pre((oop *) dest, count, /*dest_uninitialized*/true); - Copy::arrayof_conjoint_oops(src, dest, count); - gen_arraycopy_barrier((oop *) dest, count); + HeapAccess::oop_arraycopy(NULL, NULL, src, dest, count); JRT_END address StubRoutines::select_fill_function(BasicType t, bool aligned, const char* &name) {