Changeset 7fdae38

Timestamp:

Aug 14, 2020, 3:03:05 PM (4 years ago)

Author:

Thierry Delisle <tdelisle@…>

Branches:

ADT, arm-eh, ast-experimental, enum, forall-pointer-decay, jacob/cs343-translation, master, new-ast, new-ast-unique-expr, pthread-emulation, qualifiedEnum

Children:

2fafe7e

Parents:

4998155

Message:

Replaced unnecessary thread fence with sufficient compiler fence

File:

• libcfa/src/concurrency/kernel.cfa (modified) (1 diff)

libcfa/src/concurrency/kernel.cfa

@@ -532 +532 @@
                 unsigned total   = this.total;
                 processor * proc = &this.list`first;
-                // Thread fence is unnecessary, but gcc-8 and older incorrectly reorder code without it
-                __atomic_thread_fence(__ATOMIC_SEQ_CST);
+                // Compiler fence is unnecessary, but gcc-8 and older incorrectly reorder code without it
+                asm volatile("": : :"memory");
                 if(l != __atomic_load_n(&this.lock, __ATOMIC_SEQ_CST)) { Pause(); continue; }
                 return [idle, total, proc];