Commit 49b5511b authored by James Zern's avatar James Zern

Revert "configure: enable -Wshorten-64-to-32 for hbd"

This reverts commit a8b97eb5.

Reason for revert: There are still a few warnings related to this in 
the base configuration. Reverting this to avoid breaking new patches.

Change-Id: Ifdff76683cdd4b7476fcc9fbf4ab1736c112770c
parent a8b97eb5
......@@ -136,9 +136,9 @@ else ()
add_compiler_flag_if_supported("-Wuninitialized")
add_compiler_flag_if_supported("-Wunused")
add_compiler_flag_if_supported("-Wvla")
# 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")
if (NOT CONFIG_HIGHBITDEPTH)
add_c_flag_if_supported("-Wshorten-64-to-32")
endif ()
# Add -Wshadow only for C files to avoid massive gtest warning spam.
add_c_flag_if_supported("-Wshadow")
......
......@@ -722,9 +722,11 @@ process_toolchain() {
fi
# gtest makes heavy use of undefined pre-processor symbols
check_cflags "-Wundef" && add_cflags_only "-Wundef"
# 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
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
fi
if enabled icc; then
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment