On Thu, Aug 11, 2011 at 11:36 AM, arrowdodger <span dir="ltr"><<a href="mailto:6yearold@gmail.com" target="_blank">6yearold@gmail.com</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


<div><div></div><div>On Sat, Aug 6, 2011 at 2:21 PM, arrowdodger <span dir="ltr"><<a href="mailto:6yearold@gmail.com" target="_blank">6yearold@gmail.com</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">



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.<br>




<br>This patch do following things:<br>1. Rename LLVM_MULTITHREADED to LLVM_HAVE_ATOMICS in CMake checking code and <a href="http://configure.ac" target="_blank">configure.ac</a>, fix description of this option.<br>2. Replaces all uses of old define with new one.<br>




3. Fix lib/Support/Threading.cpp file by using ENABLE_THREADS define instead of LLVM_MULTITHREADED.<br>
</blockquote></div><br>
</div></div></blockquote></div><br>