Changeset dab09ad for libcfa/src/concurrency

Timestamp:

Aug 12, 2020, 1:33:00 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:

cd02108, fb0ae06

Parents:

26ffcda

Message:

Added unnecessary thread-fence to work around incorrect reorder

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);
                 if(l != __atomic_load_n(&this.lock, __ATOMIC_SEQ_CST)) { Pause(); continue; }
                 return [idle, total, proc];