diff options
| author | Enji Cooper <yaneurabeya@gmail.com> | 2019-02-19 03:19:34 (GMT) |
|---|---|---|
| committer | Enji Cooper <yaneurabeya@gmail.com> | 2019-02-19 03:19:34 (GMT) |
| commit | fcf59ca7bf6346ec08ed8591973517cdadc8e342 (patch) | |
| tree | 23183b2a3bd3907939665cccaf03d97870b51076 | |
| parent | d850e144710e330070b756c009749dc7a7302301 (diff) | |
| download | googletest-fcf59ca7bf6346ec08ed8591973517cdadc8e342.zip googletest-fcf59ca7bf6346ec08ed8591973517cdadc8e342.tar.gz googletest-fcf59ca7bf6346ec08ed8591973517cdadc8e342.tar.bz2 | |
Ignore `-Wsign-conversion` issuesrefs/pull/2137/head
clang++ compilation when `-Wsign-conversion` is currently broken and the
issues within the code are varied and widespread. For the time being
ignore `-Wsign-conversion` issues, even though some of them are valid
and bleed over into issues that would be found with
`-Wtautological-compare`, et al.
Signed-off-by: Enji Cooper <yaneurabeya@gmail.com>
| -rw-r--r-- | googletest/cmake/internal_utils.cmake | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/googletest/cmake/internal_utils.cmake b/googletest/cmake/internal_utils.cmake index 1909faf..3023904 100644 --- a/googletest/cmake/internal_utils.cmake +++ b/googletest/cmake/internal_utils.cmake @@ -78,7 +78,7 @@ macro(config_compiler_and_linker) # http://stackoverflow.com/questions/3232669 explains the issue. set(cxx_base_flags "${cxx_base_flags} -wd4702") elseif (CMAKE_CXX_COMPILER_ID STREQUAL "Clang") - set(cxx_base_flags "-Wall -Wshadow -Werror") + set(cxx_base_flags "-Wall -Wshadow -Werror -Wno-error=sign-conversion") set(cxx_exception_flags "-fexceptions") set(cxx_no_exception_flags "-fno-exceptions") elseif (CMAKE_COMPILER_IS_GNUCXX) |