Commit a8b97eb5 authored by James Zern's avatar James Zern
Browse files

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

Change-Id: I98e41e2cf58fa698ab2ec91536e82368bb04b3d7
parent 34939825
......@@ -136,9 +136,9 @@ else ()
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")
......
......@@ -722,11 +722,9 @@ process_toolchain() {
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
......
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