search

sandialabs / qthreads

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

Skip Concurrent WriteFF Test When Thread Sanitizer Is Enabled #222

Closed insertinterestingnamehere closed 9 months ago

insertinterestingnamehere commented 9 months ago

This is an alternative to https://github.com/sandialabs/qthreads/pull/221.

insertinterestingnamehere commented 9 months ago

A few of the cirrus builds are showing up as cancelled since I switched the base branch after creating the pull request. It cancelled the builds but didn't link the new ones into the checks section of the PR. They're visible at: https://cirrus-ci.com/build/6733303970004992.

insertinterestingnamehere commented 9 months ago

Looks like there's some kind of inconsistency between clang and gcc with the macro idiom I used for detecting thread sanitizer. Investigating how to work around that.

insertinterestingnamehere commented 9 months ago

Okay, that fixed it. This is good to go now.