<div dir="ltr">Hans,<div><br></div><div>Can you please work on the driver support today?</div><div><br></div><div>It's 10pm here and I don't want to delay this review too much...</div><div><br></div></div><div class="gmail_extra">

<br><br><div class="gmail_quote">2014-05-12 21:57 GMT+04:00 Reid Kleckner <span dir="ltr"><<a href="mailto:rnk@google.com" target="_blank">rnk@google.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div class="">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><div>I'd just do it first.  We basically add things as needed, as long as they don't conflict.</div><div class=""><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></div><br></div></div>
</blockquote></div><br></div>