From 253eb7416421661873afbaa33828a850db978541 Mon Sep 17 00:00:00 2001 From: Loo Rong Jie Date: Sat, 23 Mar 2019 03:23:01 +0800 Subject: [CMake] Set correct flags for clang-cl (#278) clang-cl produce binaries with MSVC ABI and wants to be as flag-compatible with pure MSVC as possible, so this leads to all sorts of weird cases. clang-cl alias /Wall as clang's -Weverything which is way too verbose, so it needs /W3 like pure MSVC. clang-cl only understand GCC style warning flags (-W[no]blah) and just silent drop MSVC style warning flags (/wd[num]). clang-cl needs MSVC define flags since it is consuming the same header files as pure MSVC. CMake set CMAKE_CXX_COMPILER_ID as Clang when clang-cl is detected, so need extra if (MSVC) to differentiate it. We are not doing clang-cl specialization in Bazel as currently there is no reliable way to detect clang-cl in Bazel.. This PR should be NFC for LLVM/GCC users on Unix platforms. Other changes: Add ABSL_ prefix to variable names to avoid name collision in CMake. --- .gitignore | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to '.gitignore') diff --git a/.gitignore b/.gitignore index 31f3374..d54fa5a 100644 --- a/.gitignore +++ b/.gitignore @@ -11,5 +11,5 @@ CMakeLists.txt.user # Ignore VS Code files .vscode/* # Ignore generated python artifacts -copts/copts.pyc +*.pyc copts/__pycache__/ -- cgit v1.2.3