melver added inline comments.
================
Comment at: clang/test/CodeGen/X86/sanitize-thread-disable.c:22
int instrumented1(int *a, _Atomic int *b) {
return *a + atomic_load(b);
}
----------------
melver wrote:
> I think you do not need to use atomic_load.
>
> You can just deref b, and because it's _Atomic type it *should* just use an
> atomic seq_cst load implicitly.
Alternatively, there are the `__atomic` builtins. You could just use
`__atomic_load_n()` instead (it works with either _Atomic or non-_Atomic type).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D108555/new/
https://reviews.llvm.org/D108555
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
