openharmony/arkcompiler_runtime_core
1
0
Fork 0
mirror of https://gitee.com/openharmony/arkcompiler_runtime_core synced 2024-11-24 07:10:22 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Fix ETS Array copy

Browse Source 
Description:
Fix incorrect copy range and GC barrier call

Issue: #IAXXR2

Signed-off-by: Panferov Ivan <panferov.ivan@huawei-partners.com>
This commit is contained in:
Panferov Ivan 2024-10-17 21:49:22 +08:00
parent 5a999f1600
commit a42e25dfb1
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
static_core/plugins/ets/runtime/types/ets_array.h
View File

@ -178,7 +178,7 @@ public:
// PreBarrier isn't needed as links inside the source object arn't changed.
auto *barrierSet = ManagedThread::GetCurrent()->GetBarrierSet();
if (!mem::IsEmptyBarrier(barrierSet->GetPostType())) {
barrierSet->PostBarrier(dst, 0, dst->ObjectSize());
barrierSet->PostBarrier(dst, GetDataOffset(), count);
}
}
}
@ -211,7 +211,7 @@ private:
reinterpret_cast<AtomicWord *>(&dst[i])->store(reinterpret_cast<AtomicWord *>(&src[i])->load(ORDER), ORDER);
}
// 2. copy by references if any
stop = ((src.Size() - i) / OBJECT_POINTER_SIZE) * OBJECT_POINTER_SIZE;
stop = src.Size();
for (; i < stop; i += OBJECT_POINTER_SIZE) {
// Atomic with parameterized order reason: memory order defined as constexpr
reinterpret_cast<AtomicRef *>(&dst[i])->store(reinterpret_cast<AtomicRef *>(&src[i])->load(ORDER), ORDER);