Reapply "Try enabling -Wsuggest-override again, using add_compile_options instead of add_compile_definitions for disabling it in unittests/ directories."

add_compile_options is more sensitive to its location in the file than add_definitions--it only takes effect for sources that are added after it. This updated patch ensures that the add_compile_options is done before adding any source files that depend on it.

Using add_definitions caused the flag to be passed to rc.exe on Windows and thus broke Windows builds.

GitOrigin-RevId: 77e0e9e17daf0865620abcd41f692ab0642367c4
1 file changed
tree: 2cbb046695310f5b6866013256c8440b171e9ea2
  1. acxxel/
  2. .clang-format
  3. .clang-tidy
  4. CMakeLists.txt
  5. README.rst