<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, May 12, 2014 at 10:50 AM, 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">-Xclang is horrible. Please add a FIXME assigned to you to remove this as soon as the necessary flags are supported by clang-cl.<br>
</blockquote><div><br></div><div>I'd just do it first.  We basically add things as needed, as long as they don't conflict.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

================<br>
Comment at: lib/asan/tests/CMakeLists.txt:77<br>
@@ -50,2 +76,3 @@<br>
 # Unit tests require libstdc++.<br>
-set(ASAN_UNITTEST_COMMON_LINKFLAGS -lstdc++)<br>
+if(NOT MSVC)<br>
+  set(ASAN_UNITTEST_COMMON_LINKFLAGS -lstdc++)<br>
----------------<br>
Why do you need this? If there is no libstdc++ on Windows, you should instead detect this in the build system and introduce<br>
COMPILER_RT_HAS_LIBSTDCXX like we do for libpthread and libdl.<br>
<br>
================<br>
Comment at: lib/asan/tests/CMakeLists.txt:106<br>
@@ -76,3 +105,3 @@<br>
<br>
-set(ASAN_UNITTEST_NOINST_LINKFLAGS<br>
-  ${ASAN_UNITTEST_COMMON_LINKFLAGS} -lm)<br>
+if(NOT MSVC)<br>
+  set(ASAN_UNITTEST_NOINST_LINKFLAGS<br>
----------------<br>
ditto<br>
<br>
<a href="http://reviews.llvm.org/D3720" target="_blank">http://reviews.llvm.org/D3720</a><br>
<br>
<br>
</blockquote></div><br></div></div>