Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(318)

Unified Diff: build/common.gypi

Issue 11468005: clang: Build in C++11 mode on linux & cros too. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: asan in, cros out Created 7 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build/common.gypi
diff --git a/build/common.gypi b/build/common.gypi
index 8f479d71a3977abef592bd781e8232eb0b972075..fe4ecbc0edbf08fe8e7bb628300a5bd9b281fb3b 100644
--- a/build/common.gypi
+++ b/build/common.gypi
@@ -3003,19 +3003,12 @@
['clang==1', {
'cflags': [
'-Wheader-hygiene',
- # Clang spots more unused functions.
- '-Wno-unused-function',
+
# Don't die on dtoa code that uses a char as an array index.
'-Wno-char-subscripts',
- # Especially needed for gtest macros using enum values from Mac
- # system headers.
- # TODO(pkasting): In C++11 this is legal, so this should be
- # removed when we change to that. (This is also why we don't
- # bother fixing all these cases today.)
- '-Wno-unnamed-type-template-args',
- # This (rightfully) complains about 'override', which we use
- # heavily.
- '-Wno-c++11-extensions',
+
+ # Clang spots more unused functions.
+ '-Wno-unused-function',
# Warns on switches on enums that cover all enum values but
# also contain a default: branch. Chrome is full of that.
@@ -3029,6 +3022,43 @@
'-mfpmath=sse',
],
}],
+ ['clang==1 and (OS!="android" and chromeos!=1)', {
+ # Turn on C++11.
+ 'cflags': [
+ # This warns on using ints as initializers for floats in
+ # initializer lists (e.g. |int a = f(); CGSize s = { a, a };|),
+ # which happens in several places in chrome code. Not sure if
+ # this is worth fixing.
+ '-Wno-c++11-narrowing',
+
+ # This warns about code like |"0x%08"NACL_PRIxPTR| -- with C++11
+ # user-defined literals, this is now a string literal with a UD
+ # suffix. However, this is used heavily in NaCl code, so disable
+ # the warning for now.
+ '-Wno-reserved-user-defined-literal',
+ ],
+ 'cflags_cc': [
+ # See the comment in the Mac section for what it takes to move
+ # this to -std=c++11.
+ '-std=gnu++11',
tfarina 2013/05/01 23:44:55 looks like this broke my build, and the instructio
+ ],
+ }],
+ ['clang==1 and (OS=="android" or chromeos==1)', {
+ # Android uses gcc4.4, and clang isn't compatible with gcc4.4's
+ # libstdc++ in C++11 mode. So no C++11 mode for Android yet.
+ # Doesn't work with asan for some reason either: crbug.com/233464
+ 'cflags': [
+ # Especially needed for gtest macros using enum values from Mac
+ # system headers.
+ # TODO(pkasting): In C++11 this is legal, so this should be
+ # removed when we change to that. (This is also why we don't
+ # bother fixing all these cases today.)
+ '-Wno-unnamed-type-template-args',
+ # This (rightfully) complains about 'override', which we use
+ # heavily.
+ '-Wno-c++11-extensions',
+ ],
+ }],
['clang==1 and clang_use_chrome_plugins==1', {
'cflags': [
'<@(clang_chrome_plugins_flags)',
@@ -3665,6 +3695,7 @@
# Don't die on dtoa code that uses a char as an array index.
# This is required solely for base/third_party/dmg_fp/dtoa.cc.
'-Wno-char-subscripts',
+
# Clang spots more unused functions.
'-Wno-unused-function',
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698