compiler flags for use in production

DrTimothyAldenDavis · Feb 23, 2025 · d94e06c · d94e06c
1 parent be15ce2
commit d94e06c
Showing 1 changed file with 4 additions and 5 deletions.
diff --git a/cmake_modules/GraphBLAS_compiler_options.cmake b/cmake_modules/GraphBLAS_compiler_options.cmake
@@ -26,9 +26,8 @@ if ( "${CMAKE_C_COMPILER_ID}" STREQUAL "GNU" )
     set ( CMAKE_C_FLAGS  "${CMAKE_C_FLAGS} -fno-math-errno " )
     # integer operations wrap
     set ( CMAKE_C_FLAGS  "${CMAKE_C_FLAGS} -fwrapv " )
-    # check all warnings (uncomment for development only)
-    # FIXME
-    set ( CMAKE_C_FLAGS  "${CMAKE_C_FLAGS} -Wall -Wextra -Wpedantic -Werror" )
+    # check all warnings (uncomment for development only):
+#   set ( CMAKE_C_FLAGS  "${CMAKE_C_FLAGS} -Wall -Wextra -Wpedantic -Werror" )
 #   set ( CMAKE_C_FLAGS  "${CMAKE_C_FLAGS} -Wall -Wextra -Wpedantic" )
     if ( CMAKE_C_COMPILER_VERSION VERSION_LESS 4.9 )
         message ( FATAL_ERROR "gcc version must be at least 4.9" )
@@ -55,8 +54,8 @@ elseif ( "${CMAKE_C_COMPILER_ID}" STREQUAL "IntelLLVM" )
 elseif ( "${CMAKE_C_COMPILER_ID}" STREQUAL "Clang" )
     # options for clang
     set ( CMAKE_C_FLAGS  "${CMAKE_C_FLAGS} -Wno-pointer-sign " )
-    # FIXME
-    set ( CMAKE_C_FLAGS  "${CMAKE_C_FLAGS} -Wall " )
+    # check all warnings (uncomment for development only):
+#   set ( CMAKE_C_FLAGS  "${CMAKE_C_FLAGS} -Wall " )
     if ( CMAKE_C_COMPILER_VERSION VERSION_LESS 3.3 )
         message ( FATAL_ERROR "clang version must be at least 3.3" )
     endif ( )