[compiler-rt] [compiler-rt][builtins] Switch libatomic locks to pthread_mutex_t. (PR #94374)
via llvm-commits
llvm-commits at lists.llvm.org
Wed Jun 5 10:58:08 PDT 2024
appujee wrote:
> This also broke builds for Windows: https://github.com/mstorsjo/llvm-mingw/actions/runs/9376329007/job/25816083980
>
> > Can you specialize this change to fall back to `_Atomic` when `COMPILER_RT_HAS_LIBPTHREAD` is off?
>
> This sounds like a good idea, but I'd also like to make it dependent on platform. E.g. on Windows, libpthread may exist - there are third party libraries that provide the pthreads API, but we don't want to use it here even if it happens to exist while building compiler-rt.
Yeah, i agree that users should have a choice to opt-in to pthreads. As it seems like there are a few things to clarify, should this patch be reverted?
https://github.com/llvm/llvm-project/pull/94374
More information about the llvm-commits
mailing list