Disabled strict aliasing by default (vesoft-inc#1665)

dutor · web-flow · commit 25b88b4fe179 · 2020-04-07T16:06:56.000+08:00
diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -53,6 +53,7 @@ option(ENABLE_COVERAGE "Whether to turn unit test coverage ON or OFF" OFF)
 option(ENABLE_UBSAN "Whether to turn Undefined Behavior Sanitizer ON or OFF" OFF)
 option(ENABLE_FUZZ_TEST "Whether to turn Fuzz Test ON or OFF" OFF)
 option(ENABLE_WERROR "Whether to error on warnings" ON)
+option(ENABLE_STRICT_ALIASING "Whether to turn strict-aliasing ON or OFF" OFF)
 option(ENABLE_TSAN "Whether to turn Thread Sanitizer ON or OFF" OFF)
 option(ENABLE_STATIC_ASAN "Whether directs the GCC driver to link libasan statically" OFF)
 option(ENABLE_STATIC_UBSAN "Whether directs the GCC driver to link libubsan statically" OFF)
@@ -65,9 +66,8 @@ message(STATUS "ENABLE_FUZZ_TEST: ${ENABLE_FUZZ_TEST}")
 message(STATUS "ENABLE_TSAN: ${ENABLE_TSAN}")
 message(STATUS "ENABLE_STATIC_ASAN : ${ENABLE_STATIC_ASAN}")
 message(STATUS "ENABLE_STATIC_UBSAN : ${ENABLE_STATIC_UBSAN}")
-
-
-add_compile_options(-fno-strict-aliasing)
+message(STATUS "ENABLE_WERROR: ${ENABLE_WERROR}")
+message(STATUS "ENABLE_STRICT_ALIASING: ${ENABLE_STRICT_ALIASING}")
 
 if (ENABLE_NATIVE)
     message(STATUS "ENABLE_NATIVE is ${ENABLE_NATIVE}")
@@ -275,6 +275,10 @@ if(ENABLE_WERROR)
     add_compile_options(-Werror)
 endif()
 
+if(NOT ENABLE_STRICT_ALIASING)
+    add_compile_options(-fno-strict-aliasing)
+endif()
+
 include(CheckCXXCompilerFlag)
 
 # TODO(dutor) Move all compiler-config-related operations to a separate file
