[llvm-commits] [PATCH] Rename LLVM_MULTITHREADED define and fix build without threads

[arrowdodger]

On Sat, Aug 6, 2011 at 2:21 PM, arrowdodger <6yearold at gmail.com> wrote:

> Hi, there is LLVM_MULTITHREADED define which is actually indicates presence
> of atomic builtins in the host system. Such name may confuse developers in
> future and already confused someone at lib/Support/Threading.cpp - he used
> this define to check whether LLVM is built with threading support. This
> mistake is currently blocking single-threaded build on Unix.
>
> This patch do following things:
> 1. Rename LLVM_MULTITHREADED to LLVM_HAVE_ATOMICS in CMake checking code
> and configure.ac, fix description of this option.
> 2. Replaces all uses of old define with new one.
> 3. Fix lib/Support/Threading.cpp file by using ENABLE_THREADS define
> instead of LLVM_MULTITHREADED.
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20110811/cc90ce12/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: llvm.atomics.define.diff
Type: text/x-patch
Size: 6430 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20110811/cc90ce12/attachment.bin>