X-Git-Url: https://git.ao2.it/libam7xxx.git/blobdiff_plain/3245aa702260b7ca45cef26622d9a90f6e602506..4f94fea5cc5a32fcb60ea070fd3a63e1b981c4a9:/CMakeLists.txt

diff --git a/CMakeLists.txt b/CMakeLists.txt
index 8264845..df06d81 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -31,7 +31,9 @@ macro(add_flags var)
   set(${var} "${${var}} ${_flags}")
 endmacro(add_flags)
 
-if (CMAKE_COMPILER_IS_GNUCC)
+string(REGEX MATCH "clang" CMAKE_COMPILER_IS_CLANG "${CMAKE_C_COMPILER}")
+
+if (CMAKE_COMPILER_IS_GNUCC OR CMAKE_COMPILER_IS_CLANG)
   add_definitions(-Wall)
 
   # let CFLAGS env override this
@@ -63,10 +65,8 @@ if (CMAKE_COMPILER_IS_GNUCC)
     -Wswitch-enum
     -Wundef
     -Wunreachable-code
-    -Wunsafe-loop-optimizations
     -Wwrite-strings
-    -fstack-protector
-    --param=ssp-buffer-size=4)
+    -fstack-protector)
 
   add_flags(DEBUG_FLAGS
     -ggdb
@@ -78,11 +78,30 @@ if (CMAKE_COMPILER_IS_GNUCC)
   if (STRICT_COMPILATION_CHECKS)
     add_flags(STRICT_FLAGS
       -Werror
+      # sign conversion warnings can be very noisy for a very little gain
+      #-Wsign-conversion
       # NOTE: Vanilla libusb-1.0.8 can't live with -pedantic-errors
-      -pedantic-errors
+      -pedantic-errors)
+
+  endif()
+endif()
+
+if (CMAKE_COMPILER_IS_GNUCC)
+  add_flags(CMAKE_C_FLAGS
+    -Wunsafe-loop-optimizations
+    --param=ssp-buffer-size=4)
+
+  if (STRICT_COMPILATION_CHECKS)
+    add_flags(STRICT_FLAGS
       # NOTE: GCC >= 4.6 is needed for -Wunused-but-set-variable
       -Wunused-but-set-variable)
+  endif()
+endif()
 
+if (CMAKE_COMPILER_IS_CLANG)
+  if (STRICT_COMPILATION_CHECKS)
+    add_flags(STRICT_FLAGS
+      -Wshorten-64-to-32)
   endif()
 endif()