fix annotate order in atomic load/store

Author: bratpiorka

src/utils/utils_posix_concurrency.c

@@ -73,23 +73,23 @@ int utils_write_unlock(utils_rwlock_t *rwlock) {
 }
 
 void utils_atomic_load_acquire_u64(uint64_t *ptr, uint64_t *out) {
-    utils_annotate_acquire(ptr);
     __atomic_load(ptr, out, memory_order_acquire);
+    utils_annotate_acquire(ptr);
 }
 
 void utils_atomic_load_acquire_ptr(void **ptr, void **out) {
-    utils_annotate_acquire((void *)ptr);
     *out = (void *)__atomic_load_n((uintptr_t *)ptr, memory_order_acquire);
+    utils_annotate_acquire((void *)ptr);
 }
 
 void utils_atomic_store_release_u64(uint64_t *ptr, uint64_t *val) {
-    __atomic_store(ptr, val, memory_order_release);
     utils_annotate_release(ptr);
+    __atomic_store(ptr, val, memory_order_release);
 }
 
 void utils_atomic_store_release_ptr(void **ptr, void *val) {
-    __atomic_store_n((uintptr_t *)ptr, (uintptr_t)val, memory_order_release);
     utils_annotate_release(ptr);
+    __atomic_store_n((uintptr_t *)ptr, (uintptr_t)val, memory_order_release);
 }
 
 uint64_t utils_atomic_increment_u64(uint64_t *val) {

src/utils/utils_windows_concurrency.c

@@ -72,29 +72,29 @@ void utils_atomic_load_acquire_u64(uint64_t *ptr, uint64_t *out) {
     // NOTE: Windows cl complains about direct accessing 'ptr' which is next
     // accessed using Interlocked* functions (warning 28112 - disabled)
     ASSERT_IS_ALIGNED((uintptr_t)ptr, 8);
-    utils_annotate_acquire(ptr);
     LONG64 ret = InterlockedCompareExchange64((LONG64 volatile *)ptr, 0, 0);
+    utils_annotate_acquire(ptr);
     *out = *(uint64_t *)&ret;
 }
 
 void utils_atomic_load_acquire_ptr(void **ptr, void **out) {
     ASSERT_IS_ALIGNED((uintptr_t)ptr, 8);
-    utils_annotate_acquire((void *)ptr);
     uintptr_t ret = (uintptr_t)InterlockedCompareExchangePointer(ptr, 0, 0);
+    utils_annotate_acquire((void *)ptr);
     *(uintptr_t *)out = ret;
 }
 
 void utils_atomic_store_release_u64(uint64_t *ptr, uint64_t *val) {
     ASSERT_IS_ALIGNED((uintptr_t)ptr, 8);
     ASSERT_IS_ALIGNED((uintptr_t)val, 8);
-    InterlockedExchange64((LONG64 volatile *)ptr, *(LONG64 *)val);
     utils_annotate_release(ptr);
+    InterlockedExchange64((LONG64 volatile *)ptr, *(LONG64 *)val);
 }
 
 void utils_atomic_store_release_ptr(void **ptr, void *val) {
     ASSERT_IS_ALIGNED((uintptr_t)ptr, 8);
-    InterlockedExchangePointer(ptr, val);
     utils_annotate_release(ptr);
+    InterlockedExchangePointer(ptr, val);
 }
 
 uint64_t utils_atomic_increment_u64(uint64_t *ptr) {