[PATCH] Move duplicate Windows-specific compiler flags to a common CMake variable

[Timur Iskhodzhanov]

Can you elaborate on the -isystem approach? I'm not sure I know what you're
taking about.
29 мая 2014 г. 21:03 пользователь "Reid Kleckner" <rnk at google.com> написал:

> drive by nits that were already there
>
> ================
> Comment at: cmake/Modules/AddCompilerRT.cmake:129
> @@ -126,1 +128,3 @@
>  if(MSVC)
> +  # MSVC system headers and gtest use a lot of deprecated stuff.
> +  list(APPEND COMPILER_RT_TEST_CFLAGS
> ----------------
> Can we use -isystem to suppress these?  Does -D_CRT_SECURE_NO_WARNINGS
> solve this problem?
>
> ================
> Comment at: cmake/Modules/AddCompilerRT.cmake:137
> @@ +136,3 @@
> +
> +  # We should teach clang to understand more pragmas.
> +  list(APPEND COMPILER_RT_TEST_CFLAGS
> ----------------
> ditto, this can be handled with -isystem.
>
> http://reviews.llvm.org/D3952
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140529/a556bf20/attachment.html>