Skip to content

A
angle
Commit 74f44506 by Jamie Madill
Options

Only use the shortening warning in Clang.

This seems to be Clang-only. Change-Id: I4276a29e3d1575bd47d58d819ed3223b2c66838a Reviewed-on: https://chromium-review.googlesource.com/321820Reviewed-by: 's avatarCorentin Wallez <cwallez@chromium.org> Tested-by: 's avatarJamie Madill <jmadill@chromium.org>
parent adc5b0eb
Showing with 9 additions and 1 deletion
build/common.gypi
...@@ -31,13 +31,17 @@ ...@@ -31,13 +31,17 @@
'-Wno-unused-parameter', '-Wno-unused-parameter',
'-Wpacked', '-Wpacked',
'-Wpointer-arith', '-Wpointer-arith',
'-Wshorten-64-to-32',
'-Wundef', '-Wundef',
'-Wwrite-strings', '-Wwrite-strings',
], ],
# TODO: Pull chromium's clang dep. # TODO: Pull chromium's clang dep.
'clang%': 0, 'clang%': 0,
'clang_only_warnings':
[
'-Wshorten-64-to-32',
],
}, },
'target_defaults': 'target_defaults':
{ {
...@@ -439,6 +443,10 @@ ...@@ -439,6 +443,10 @@
{ {
'cflags': ['<@(gcc_or_clang_warnings)'] 'cflags': ['<@(gcc_or_clang_warnings)']
}], }],
['clang==1',
{
'cflags': ['<@(clang_only_warnings)']
}],
] ]
} }
}], }],
......
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