configure: enable -Wshorten-64-to-32 for hbd
Change-Id: I98e41e2cf58fa698ab2ec91536e82368bb04b3d7
diff --git a/build/cmake/aom_configure.cmake b/build/cmake/aom_configure.cmake
index 6a1fe13..78426f2 100644
--- a/build/cmake/aom_configure.cmake
+++ b/build/cmake/aom_configure.cmake
@@ -136,9 +136,9 @@
add_compiler_flag_if_supported("-Wuninitialized")
add_compiler_flag_if_supported("-Wunused")
add_compiler_flag_if_supported("-Wvla")
- if (NOT CONFIG_HIGHBITDEPTH)
- add_c_flag_if_supported("-Wshorten-64-to-32")
- endif ()
+ # TODO(jzern): this could be added as a cxx flags for test/*.cc only,
+ # avoiding third_party.
+ add_c_flag_if_supported("-Wshorten-64-to-32")
# Add -Wshadow only for C files to avoid massive gtest warning spam.
add_c_flag_if_supported("-Wshadow")
diff --git a/configure b/configure
index c741f7b..5e105b9 100755
--- a/configure
+++ b/configure
@@ -722,11 +722,9 @@
fi
# gtest makes heavy use of undefined pre-processor symbols
check_cflags "-Wundef" && add_cflags_only "-Wundef"
- if ! enabled highbitdepth; then
- # Avoid this warning for third_party C++ sources. Some reorganization
- # would be needed to apply this only to test/*.cc.
- check_cflags -Wshorten-64-to-32 && add_cflags_only -Wshorten-64-to-32
- fi
+ # Avoid this warning for third_party C++ sources. Some reorganization
+ # would be needed to apply this only to test/*.cc.
+ check_cflags -Wshorten-64-to-32 && add_cflags_only -Wshorten-64-to-32
fi
if enabled icc; then
