[Openmp-commits] [PATCH] D65867: [RFC] [OpenMP] Turn on -Wall compiler warnings by default

[Jonathan Peyton via Phabricator via Openmp-commits]

jlpeyton added a comment.

Thanks for this cleanup! Just one comment.



================
Comment at: openmp/cmake/config-ix.cmake:4-15
+check_c_compiler_flag(-Wall OPENMP_HAVE_WALL_FLAG)
 check_c_compiler_flag(-Werror OPENMP_HAVE_WERROR_FLAG)
 
+# Additional warnings that are not enabled by -Wall.
+check_cxx_compiler_flag(-Wcast-qual OPENMP_HAVE_WCAST_QUAL_FLAG)
+check_c_compiler_flag(-Wformat-pedantic OPENMP_HAVE_WFORMAT_PEDANTIC_FLAG)
+check_c_compiler_flag(-Wsign-compare OPENMP_HAVE_WSIGN_COMPARE_FLAG)
----------------
Should everything be check_cxx_compiler_flag()?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D65867/new/

https://reviews.llvm.org/D65867