Skip to content

Commit 34f66e5

Browse files
authored
Merge pull request #5894 from martin-frbg/getrf_tsan
Change atomic semantics in GETRF to acquire/release to fix tsan warnings
2 parents 71e999b + 57da6f5 commit 34f66e5

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

lapack/getrf/getrf_parallel.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -69,8 +69,8 @@ double sqrt(double);
6969

7070

7171
#ifdef HAVE_C11
72-
#define atomic_load_long(p) __atomic_load_n(p, __ATOMIC_RELAXED)
73-
#define atomic_store_long(p, v) __atomic_store_n(p, v, __ATOMIC_RELAXED)
72+
#define atomic_load_long(p) __atomic_load_n(p, __ATOMIC_ACQUIRE)
73+
#define atomic_store_long(p, v) __atomic_store_n(p, v, __ATOMIC_RELEASE)
7474
#else
7575
#define atomic_load_long(p) (BLASLONG)(*(volatile BLASLONG*)(p))
7676
#define atomic_store_long(p, v) (*(volatile BLASLONG *)(p)) = (v)

0 commit comments

Comments
 (0)