Clang: enable -Wtautological-constant-out-of-range-compare
This warning was implemented in Clang a couple of weeks ago, and now our code base is clean enough to enable it. BUG=151927 Review URL: https://chromiumcodereview.appspot.com/11366060 git-svn-id: http://src.chromium.org/svn/trunk/src/build@165793 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
This commit is contained in:
Родитель
7252b31adc
Коммит
c2a330f653
13
common.gypi
13
common.gypi
|
@ -2525,22 +2525,12 @@
|
||||||
# Warns on switches on enums that cover all enum values but
|
# Warns on switches on enums that cover all enum values but
|
||||||
# also contain a default: branch. Chrome is full of that.
|
# also contain a default: branch. Chrome is full of that.
|
||||||
'-Wno-covered-switch-default',
|
'-Wno-covered-switch-default',
|
||||||
|
|
||||||
# TODO(thakis): Remove this once http://crbug.com/151927 is fixed.
|
|
||||||
'-Wno-tautological-constant-out-of-range-compare',
|
|
||||||
],
|
],
|
||||||
'cflags!': [
|
'cflags!': [
|
||||||
# Clang doesn't seem to know know this flag.
|
# Clang doesn't seem to know know this flag.
|
||||||
'-mfpmath=sse',
|
'-mfpmath=sse',
|
||||||
],
|
],
|
||||||
}],
|
}],
|
||||||
['OS=="android"', {
|
|
||||||
'cflags!': [
|
|
||||||
# Clang ARM does not support the following option.
|
|
||||||
# TODO: Add this flag back http://crbug.com/157195.
|
|
||||||
'-Wno-tautological-constant-out-of-range-compare',
|
|
||||||
]
|
|
||||||
}],
|
|
||||||
['clang==1 and clang_use_chrome_plugins==1', {
|
['clang==1 and clang_use_chrome_plugins==1', {
|
||||||
'cflags': [
|
'cflags': [
|
||||||
'<@(clang_chrome_plugins_flags)',
|
'<@(clang_chrome_plugins_flags)',
|
||||||
|
@ -3060,9 +3050,6 @@
|
||||||
# Warns on switches on enums that cover all enum values but
|
# Warns on switches on enums that cover all enum values but
|
||||||
# also contain a default: branch. Chrome is full of that.
|
# also contain a default: branch. Chrome is full of that.
|
||||||
'-Wno-covered-switch-default',
|
'-Wno-covered-switch-default',
|
||||||
|
|
||||||
# TODO(thakis): Remove this once http://crbug.com/151927 is fixed.
|
|
||||||
'-Wno-tautological-constant-out-of-range-compare',
|
|
||||||
],
|
],
|
||||||
}],
|
}],
|
||||||
['clang==1 and clang_use_chrome_plugins==1', {
|
['clang==1 and clang_use_chrome_plugins==1', {
|
||||||
|
|
Загрузка…
Ссылка в новой задаче