<div dir="ltr">Clang suppresses all warnings in system headers, which are put onto the include search path with the -isystem flag.  In clang-cl, we consider anything in %INCLUDE% to be a system header by passing it with -isystem to clang -cc1.</div>
<div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, May 29, 2014 at 11:41 AM, Timur Iskhodzhanov <span dir="ltr"><<a href="mailto:timurrrr@google.com" target="_blank">timurrrr@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><p dir="ltr">Can you elaborate on the -isystem approach? I'm not sure I know what you're taking about.</p>
<div class="gmail_quote">29 мая 2014 г. 21:03 пользователь "Reid Kleckner" <<a href="mailto:rnk@google.com" target="_blank">rnk@google.com</a>> написал:<div><div class="h5"><br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc 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>
<br>
================<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>
<br>
<a href="http://reviews.llvm.org/D3952" target="_blank">http://reviews.llvm.org/D3952</a><br>
<br>
<br>
</blockquote></div></div></div>
</blockquote></div><br></div>