<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">2014-05-29 21:03 GMT+04:00 Reid Kleckner <span dir="ltr"><<a href="mailto:rnk@google.com" target="_blank" class="cremed">rnk@google.com</a>></span>:<br>

<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">drive by nits that were already there<br>
<br>
================<br>
Comment at: cmake/Modules/AddCompilerRT.cmake:129<br>
@@ -126,1 +128,3 @@<br>
 if(MSVC)<br>
+  # MSVC system headers and gtest use a lot of deprecated stuff.<br>
+  list(APPEND COMPILER_RT_TEST_CFLAGS<br>
----------------<br>
Can we use -isystem to suppress these?  Does -D_CRT_SECURE_NO_WARNINGS solve this problem?\<br></blockquote><div><br></div><div>-D_CRT_SECURE_NO_WARNINGS doesn't solve the problem.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">


================<br>
Comment at: cmake/Modules/AddCompilerRT.cmake:137<br>
@@ +136,3 @@<br>
+<br>
+  # We should teach clang to understand more pragmas.<br>
+  list(APPEND COMPILER_RT_TEST_CFLAGS<br>
----------------<br>
ditto, this can be handled with -isystem.<br></blockquote><div><br></div><div>In fact, the warning comes from the sanitizer runtime code.</div></div><br></div></div>