<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" class="cremed">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">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" class="cremed">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" class="cremed">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" class="cremed">http://reviews.llvm.org/D3952</a><br>
<br>
<br>
</blockquote></div></div></div>
</blockquote></div><br></div>
</div></div></blockquote></div><br></div></div></div>