From 58c1ff87c8082473e4dcdefb88745514de4cb2d4 Mon Sep 17 00:00:00 2001
From: Marko Petric <marko.petric@cern.ch>
Date: Fri, 2 Jun 2017 13:31:21 +0200
Subject: [PATCH] Separate out clang compiler flags and set them only of clang
present
---
cmake/DD4hepBuild.cmake | 7 ++++++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/cmake/DD4hepBuild.cmake b/cmake/DD4hepBuild.cmake
index 251708600..897c5aeef 100644
--- a/cmake/DD4hepBuild.cmake
+++ b/cmake/DD4hepBuild.cmake
@@ -37,7 +37,12 @@ find_package(Threads REQUIRED)
macro(dd4hep_set_compiler_flags)
include(CheckCXXCompilerFlag)
- SET(COMPILER_FLAGS -Wall -Wextra -pedantic -Wshadow -Wformat-security -Wno-long-long -Wdeprecated -fdiagnostics-color=auto -Winconsistent-missing-override)
+ SET(COMPILER_FLAGS -Wall -Wextra -pedantic -Wshadow -Wformat-security -Wno-long-long -Wdeprecated -fdiagnostics-color=auto)
+
+ # AppleClang/Clang specific warning flags
+ if(CMAKE_CXX_COMPILER_ID MATCHES "^(Apple)?Clang$")
+ set ( COMPILER_FLAGS ${COMPILER_FLAGS} -Winconsistent-missing-override -Wno-c++1z-extensions)
+ endif()
FOREACH( FLAG ${COMPILER_FLAGS} )
## meed to replace the minus or plus signs from the variables, because it is passed
--
GitLab