commit 78a154f5bae9e499a2eec2534c3b38d512277ee4
author Aleksei.Glushko <aleksei.glushko@jetbrains.com> Sun Aug 27 13:46:51 2023 +0200
committer Aleksei.Glushko <aleksei.glushko@jetbrains.com> Sun Aug 27 13:46:51 2023 +0200
tree 02c53a3d777f12e81849b8a18076b0f9a14ee105
parent 551857557c09c24a996823707498bd651b8bcba3

try another approach

kotlin-native/runtime/src/mm/cpp/Memory.cpp

diff --git a/kotlin-native/runtime/src/mm/cpp/Memory.cpp b/kotlin-native/runtime/src/mm/cpp/Memory.cpp
index d4aa263..835cf29 100644
--- a/kotlin-native/runtime/src/mm/cpp/Memory.cpp
+++ b/kotlin-native/runtime/src/mm/cpp/Memory.cpp
@@ -634,5 +634,7 @@
 }
 
 void kotlin::OnMemoryAllocation(size_t totalAllocatedBytes) noexcept {
+    // Prevents unsafe class publication (see KT-58995).
+    std::atomic_thread_fence(std::memory_order_release);
     mm::GlobalData::Instance().gcScheduler().setAllocatedBytes(totalAllocatedBytes);
 }

kotlin-native/runtime/src/mm/cpp/ObjectOps.cpp

diff --git a/kotlin-native/runtime/src/mm/cpp/ObjectOps.cpp b/kotlin-native/runtime/src/mm/cpp/ObjectOps.cpp
index e2d5ac5..7444497 100644
--- a/kotlin-native/runtime/src/mm/cpp/ObjectOps.cpp
+++ b/kotlin-native/runtime/src/mm/cpp/ObjectOps.cpp
@@ -75,8 +75,6 @@
     AssertThreadState(threadData, ThreadState::kRunnable);
     // TODO: Make this work with GCs that can stop thread at any point.
     auto* object = threadData->gc().CreateObject(typeInfo);
-    // Prevents unsafe class publication (see KT-58995).
-    std::atomic_thread_fence(std::memory_order_release);
     RETURN_OBJ(object);
 }
 
@@ -84,8 +82,6 @@
     AssertThreadState(threadData, ThreadState::kRunnable);
     // TODO: Make this work with GCs that can stop thread at any point.
     auto* array = threadData->gc().CreateArray(typeInfo, static_cast<uint32_t>(elements));
-    // Prevents unsafe class publication (see KT-58995).
-    std::atomic_thread_fence(std::memory_order_release);
     // `ArrayHeader` and `ObjHeader` are expected to be compatible.
     RETURN_OBJ(reinterpret_cast<ObjHeader*>(array));
 }