sandialabs / qthreads

Lightweight locality-aware user-level threading runtime.
https://www.sandia.gov/qthreads/
Other
169 stars 34 forks source link

Explicit Atomics in Syncvar Implementation #235

Closed insertinterestingnamehere closed 5 months ago

insertinterestingnamehere commented 7 months ago

This one's more straightforward than the debate around aligned_t because syncvars were already treated as atomics internally. This was just a matter of making the loads and stores explicit.

I also made ARM take the same branch as x86 for these blocks of code since we're currently assuming lock-free atomics there too.