turn on -Werror=return-type

Xilinx · Dec 3, 2023 · 2593c29 · 2593c29
1 parent 900efba
commit 2593c29
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -156,6 +156,8 @@ if(CMAKE_CXX_COMPILER_ID MATCHES "GNU" AND CMAKE_CXX_COMPILER_VERSION VERSION_LE
   append_if(CXX_SUPPORTS_WNO_UNUSED_BUT_SET_PARAMETER "-Wno-unused-but-set-parameter" CMAKE_CXX_FLAGS)
 endif()
 
+add_compile_options(-Werror=return-type)
+
 # Make sure we generate the headers
 add_custom_target(aie-headers)
 set_target_properties(aie-headers PROPERTIES FOLDER "Misc")

diff --git a/utils/mlir_wheels/setup.py b/utils/mlir_wheels/setup.py
@@ -94,7 +94,6 @@ def build_extension(self, ext: CMakeExtension) -> None:
             "-DLLVM_CCACHE_BUILD=ON",
             "-DLLVM_ENABLE_ASSERTIONS=ON",
             "-DLLVM_ENABLE_RTTI=ON",
-            "-DLLVM_ENABLE_WARNINGS=OFF",
             "-DLLVM_ENABLE_ZSTD=OFF",
             "-DLLVM_INCLUDE_BENCHMARKS=OFF",
             "-DLLVM_INCLUDE_EXAMPLES=OFF",