On Mon, Sep 12, 2011 at 10:09 PM, Eric Christopher <span dir="ltr"><<a href="mailto:echristo@apple.com">echristo@apple.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 style="word-wrap:break-word">Sorry for the delay. In general the patch is fine, could you change these couple of things?<div><br></div><div>LLVM_HAVE_ATOMICS -> LLVM_HAS_ATOMICS</div></div></blockquote><div><br>A lot of other defines have _HAVE_, not _HAS_, so i thought it would be consistent. Fixed, anyway.<br>

<br></div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;"><div style="word-wrap: break-word;"><div>and</div><div><br></div><div><div>-#if LLVM_MULTITHREADED==0</div>

<div>+#if LLVM_HAVE_ATOMICS==0 || !defined(LLVM_HAVE_ATOMICS)</div><div><br></div><div>to just #if LLVM_HAS_ATOMICS</div><div><br></div><div>this latter is ok because we define it either way so there's always a value at least as far as the autoconf bits are concerned. If this isn't true for the cmake bits let me know.</div>

</div></div></blockquote><div><br>I've changed config headers templates for cmake build to make them always contain such definition. <br><br>Also, i've updated configure script itself too. Patch attached.<br>Thanks for your attention to this.<br>

</div></div>