<div dir="ltr">Are all the warnings in gtest headers or in the Windows SDK or CRT?  There are lots of pragmas we don't understand in windows.h, for example.  We should already understand all of gtest, but maybe I'm wrong.<br>
<div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, May 29, 2014 at 1:35 PM, 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"><div dir="ltr">A-ha, so you suggest to use -isystem rather than -I for gtest, right?<div>The system headers should be fine out of the box already?</div>
<div>(Please note I don't use clang-cl there now)<br><div class="gmail_extra">


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