Merge "Re-enable unused-command-line-argument warning"

This commit is contained in:
Yi Kong
2021-10-13 04:12:12 +00:00
committed by Gerrit Code Review

View File

@@ -77,10 +77,6 @@ var (
// TODO: can we remove this now? // TODO: can we remove this now?
"-Wno-reserved-id-macro", "-Wno-reserved-id-macro",
// Workaround for ccache with clang.
// See http://petereisentraut.blogspot.com/2011/05/ccache-and-clang.html.
"-Wno-unused-command-line-argument",
// Force clang to always output color diagnostics. Ninja will strip the ANSI // Force clang to always output color diagnostics. Ninja will strip the ANSI
// color codes if it is not running in a terminal. // color codes if it is not running in a terminal.
"-fcolor-diagnostics", "-fcolor-diagnostics",
@@ -329,6 +325,12 @@ func init() {
// Default to zero initialization. // Default to zero initialization.
flags = append(flags, "-ftrivial-auto-var-init=zero -enable-trivial-auto-var-init-zero-knowing-it-will-be-removed-from-clang") flags = append(flags, "-ftrivial-auto-var-init=zero -enable-trivial-auto-var-init-zero-knowing-it-will-be-removed-from-clang")
} }
// Workaround for ccache with clang.
// See http://petereisentraut.blogspot.com/2011/05/ccache-and-clang.html.
if ctx.Config().IsEnvTrue("USE_CCACHE") {
flags = append(flags, "-Wno-unused-command-line-argument")
}
return strings.Join(flags, " ") return strings.Join(flags, " ")
}) })