Changeset 7fdae38


Ignore:
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:
1 edited

Legend:

Unmodified
Added
Removed
  • libcfa/src/concurrency/kernel.cfa

    r4998155 r7fdae38  
    532532                unsigned total   = this.total;
    533533                processor * proc = &this.list`first;
    534                 // Thread fence is unnecessary, but gcc-8 and older incorrectly reorder code without it
    535                 __atomic_thread_fence(__ATOMIC_SEQ_CST);
     534                // Compiler fence is unnecessary, but gcc-8 and older incorrectly reorder code without it
     535                asm volatile("": : :"memory");
    536536                if(l != __atomic_load_n(&this.lock, __ATOMIC_SEQ_CST)) { Pause(); continue; }
    537537                return [idle, total, proc];
Note: See TracChangeset for help on using the changeset viewer.