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

Unified Diff: BUILD.gn

Issue 1909273002: 🐳 Move linux pkg_config() calls into separate BUILD.gn files (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: comments & visibility Created 4 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 | « .gn ('k') | build/config/linux/BUILD.gn » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: BUILD.gn
diff --git a/BUILD.gn b/BUILD.gn
index d89ba2cf03c3985fb25302d1c9f302b8b70f4620..284d1006aa6a8101e7f5e6c7b93e3dc1d9572636 100644
--- a/BUILD.gn
+++ b/BUILD.gn
@@ -892,6 +892,24 @@ group("chromium_builder_perf") {
group("aura_builder") {
}
+if (is_android) {
+ group("optimize_gn_gen") {
+ deps = [
+ # These run expensive scripts in non-default toolchains. Generally, host
+ # toolchain targets are loaded in the later part of the run, and the
+ # result is they push out the end of generation. By preloading these, the
+ # scripts can be parallelized with the rest of the load.
+ "//build/config/linux(//build/toolchain/linux:clang_x64)",
+ "//build/config/posix(//build/toolchain/linux:clang_x64)",
+
+ # Include x86 toolchains as well since V8 uses them for 32-bit snapshot
+ # generation.
+ "//build/config/linux(//build/toolchain/linux:clang_x86)",
+ "//build/config/posix(//build/toolchain/linux:clang_x86)",
+ ]
+ }
+}
+
# Because of the source assignment filter, many targets end up over-filtering
# their sources if the output directory contains a platform name. Assert that
# this doesn't happen. http://crbug.com/548283
« no previous file with comments | « .gn ('k') | build/config/linux/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698