Commit 0c08a756 authored by Nico Weber's avatar Nico Weber Committed by Commit Bot

Remove explicit -Wextra-semi, -Winconsistent-missing-override flags.

They're  enabled via the chromium_code config already these days.
No intended behavior change.

Bug: chromium:926235,chromium:428099
Change-Id: I4f7024ab78cb9672ec3db686e0155b2d835a4790
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1564694Reviewed-by: 's avatarMichael Achenbach <machenbach@chromium.org>
Commit-Queue: Nico Weber <thakis@chromium.org>
Auto-Submit: Nico Weber <thakis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#60812}
parent c3597280
...@@ -645,15 +645,7 @@ config("toolchain") { ...@@ -645,15 +645,7 @@ config("toolchain") {
} }
if (is_clang) { if (is_clang) {
cflags += [ cflags += [ "-Wmissing-field-initializers" ]
"-Wmissing-field-initializers",
# TODO(thakis): Remove once enabled globally, https://crbug.com/926235
"-Wextra-semi",
# TODO(hans): Remove once http://crbug.com/428099 is resolved.
"-Winconsistent-missing-override",
]
if (v8_current_cpu != "mips" && v8_current_cpu != "mipsel") { if (v8_current_cpu != "mips" && v8_current_cpu != "mipsel") {
# We exclude MIPS because the IsMipsArchVariant macro causes trouble. # We exclude MIPS because the IsMipsArchVariant macro causes trouble.
......
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