<div dir="ltr">Two things:<div><br></div><div>1. Should we add -o to clang-cl?  MSVC ignores it silently, and CMake always uses it when it does its initial test compile.</div><div>2. Should we even use clang-cl when building compiler-rt?  Do we have lots of CFLAGS in compiler-rt?  We will either have to expose them as core clang-cl options, or switch to the regular clang driver.  Do you need this to handle /MT or /MD?</div>
</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, May 8, 2014 at 1:22 PM, Alexey Samsonov <span dir="ltr"><<a href="mailto:samsonov@google.com" target="_blank">samsonov@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">LGTM modulo nits.<br>
<br>
================<br>
Comment at: CMakeLists.txt:65<br>
@@ +64,3 @@<br>
<div class="">+    set(COMPILER_RT_TEST_COMPILER_EXE "-o")<br>
+    set(COMPILER_RT_TEST_COMPILER_OUT "-o")<br>
+  else()<br>
</div>----------------<br>
_OUT makes no sense to me. You can probably use EXE/OBJ instead.<br>
<br>
================<br>
Comment at: cmake/Modules/AddCompilerRT.cmake:140<br>
@@ -140,1 +139,3 @@<br>
<div class="">+    COMMAND ${COMPILER_RT_TEST_COMPILER} ${TEST_OBJECTS}<br>
+            ${COMPILER_RT_TEST_COMPILER_EXE}"${output_bin}"<br>
             ${TEST_LINK_FLAGS}<br>
</div>----------------<br>
Sanity check: is it ok to not have a space between ${COMPILER_RT_TEST_COMPILER_EXE} and "${output_bin}" ?<br>
<br>
<a href="http://reviews.llvm.org/D3680" target="_blank">http://reviews.llvm.org/D3680</a><br>
<br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
</blockquote></div><br></div>