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

Unified Diff: build/config/clang/BUILD.gn

Issue 2079283002: [iOS/GN] Allow compilation with system clang. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Address typos Created 4 years, 6 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 | « build/config/BUILDCONFIG.gn ('k') | build/config/clang/clang.gni » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build/config/clang/BUILD.gn
diff --git a/build/config/clang/BUILD.gn b/build/config/clang/BUILD.gn
index dbff398e1f6d06231396b432f7167d00bfc50e38..ea568b3da47668d6bf3538e729cf66ab4f06f9d1 100644
--- a/build/config/clang/BUILD.gn
+++ b/build/config/clang/BUILD.gn
@@ -60,6 +60,15 @@ config("find_bad_constructs") {
]
}
}
+
+ # TODO(ios): Remove once Xcode's libc++ has LLVM r256325.
+ if (use_system_clang && is_ios) {
Nico 2016/06/21 12:30:59 can we call this use_xcode_clang please? i don't w
sdefresne 2016/06/21 13:21:58 I've renamed the variable to "is_clang_xcode" to b
justincohen 2016/06/22 09:15:07 We haven't tried shipping with a chromium clang.
Nico 2016/06/22 14:30:27 The last two times (bitcode and...uh, don't rememb
+ cflags += [
+ "-isystem",
+ rebase_path("//third_party/llvm-build/Release+Asserts/include/c++/v1",
+ root_build_dir),
+ ]
+ }
}
# Enables some extra Clang-specific warnings. Some third-party code won't
« no previous file with comments | « build/config/BUILDCONFIG.gn ('k') | build/config/clang/clang.gni » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698