[Openmp-commits] [PATCH] D11253: [cmake] Support enabling -Werror in the OpenMP runtime CMake build and clean up the build.

[Jonathan Peyton]

jlpeyton added inline comments.

================
Comment at: runtime/CMakeLists.txt:44
@@ -43,1 +43,3 @@
     "enable assertions?")
+  set(LIBOMP_ENABLE_WERROR TRUE CACHE BOOL
+    "Enable -Werror flags to turn warnings into errors for supporting compilers.")
----------------
I think this should be FALSE.  Even LLVM has it off by default.  For example, on Linux building x86_64, gcc 4.8 warns that it might not be able to inline something that has an "always inline" attribute.


http://reviews.llvm.org/D11253