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

[arrowdodger]

On Thu, Aug 11, 2011 at 11:36 AM, arrowdodger <6yearold at gmail.com> wrote:

> 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/20110816/e7c39a06/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/20110816/e7c39a06/attachment.bin>