Instrumented libraries: improve parallel build

third_party/instrumented_libraries/BUILD.gn

 # Copyright 2015 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 import("//build/config/sanitizers/sanitizers.gni")
 import("//build/toolchain/goma.gni")
 
 declare_args() {
-  # The default number of jobs to use when building instrumented libraries.
-  instrumented_libraries_jobs = 8
-
   # The platform on which the instrumented libraries are being built.
   instrumented_libraries_platform = "trusty"
 }
 
 group("deps") {
   if (use_prebuilt_instrumented_libraries) {
     assert(prebuilt_instrumented_libraries_available,
            "Prebuilt instrumented libraries are only available when " +
                "is_msan = true and msan_track_origins = {0, 2}")
     deps = [
@@ skipping 173 matching lines @@
       if (defined(invoker.build_method)) {
         build_method = invoker.build_method
       }
 
       cc = rebase_path("//third_party/llvm-build/Release+Asserts/bin/clang")
       cxx = rebase_path("//third_party/llvm-build/Release+Asserts/bin/clang++")
       if (use_goma) {
         cc = "${goma_dir}/gomacc ${cc}"
         cxx = "${goma_dir}/gomacc ${cxx}"
       }
-      jobs = instrumented_libraries_jobs
-      if (defined(invoker.jobs)) {
-        jobs = invoker.jobs
-      }
       package_cflags = [
         "-O2",
         "-gline-tables-only",
         "-fPIC",
         "-w",
         "-U_FORITFY_SOURCE",
         "-fno-omit-frame-pointer",
       ]
       package_ldflags = []
       if (is_asan) {
@@ skipping 18 matching lines @@
       if (defined(invoker.deps)) {
         deps = invoker.deps
       }
       product_dir = rebase_path(root_out_dir)
       intermediate_dir = rebase_path(target_gen_dir)
       args = [
         "--build-method=${build_method}",
         "--cc=${cc}",
         "--cxx=${cxx}",
         "--intermediate-dir=${intermediate_dir}",
-        "--jobs=${jobs}",
         "--libdir=lib",
         "--package=${target_name}",
         "--product-dir=${product_dir}",
         "--cflags=${package_cflags}",
         "--ldflags=${package_ldflags}",
         "--sanitizer=${sanitizer_type}",
       ]
       outputs = [
         "${root_out_dir}/instrumented_libraries/${sanitizer_type}/${target_name}.txt",
       ]
@@ skipping 88 matching lines @@
     ]
   }
 
   instrumented_library("libappindicator1") {
     extra_configure_flags = [
       "--disable-static",
 
       # See above.
       "--disable-introspection",
     ]
-    jobs = 1
     pre_build = "scripts/pre-build/autogen.sh"
   }
 
   instrumented_library("libasound2") {
     extra_configure_flags = [ "--disable-static" ]
     pre_build = "scripts/pre-build/libasound2.sh"
   }
 
   instrumented_library("libatk-bridge2.0-0") {
     extra_configure_flags = [ "--disable-static" ]
@@ skipping 17 matching lines @@
     extra_configure_flags = [ "--disable-static" ]
     build_method = "custom_libcap"
   }
 
   instrumented_library("libcredentialkit_pkcs11") {
     build_method = "stub"
   }
 
   instrumented_library("libcups2") {
     patch = "patches/libcups2.diff"
-    jobs = 1
     extra_configure_flags = [
       "--disable-static",
 
       # All from debian/rules.
       "--localedir=/usr/share/cups/locale",
       "--enable-slp",
       "--enable-libpaper",
       "--enable-ssl",
       "--enable-gnutls",
       "--disable-openssl",
@@ skipping 211 matching lines @@
   instrumented_library("libp11-kit0") {
     extra_configure_flags = [ "--disable-static" ]
 
     # Required on Trusty due to autoconf version mismatch.
     pre_build = "scripts/pre-build/autoreconf.sh"
   }
 
   instrumented_library("libpci3") {
     extra_configure_flags = [ "--disable-static" ]
     build_method = "custom_libpci3"
-    jobs = 1
   }
 
   instrumented_library("libpcre3") {
     extra_configure_flags = [
       "--enable-utf8",
       "--enable-unicode-properties",
       "--disable-static",
     ]
   }
 
@@ skipping 165 matching lines @@
       # is a huge pain in the neck. Let's link them statically instead, and
       # hope for the best.
       "--with-included-modules=yes",
     ]
   }
 
   instrumented_library("pulseaudio") {
     # New location of libpulsecommon.
     if (is_precise) {
       patch = "patches/pulseaudio.precise.diff"
-      jobs = 1
     }
     if (is_trusty) {
       package_ldflags = [ "-Wl,-R,XORIGIN/pulseaudio/." ]
     }
     extra_configure_flags = [
       "--disable-static",
 
       # From debian/rules.
       "--enable-x11",
       "--disable-hal-compat",
@@ skipping 15 matching lines @@
       "--disable-gudev",
     ]
     pre_build = "scripts/pre-build/udev.sh"
   }
 
   instrumented_library("zlib1g") {
     # --disable-static is not supported
     patch = "patches/zlib1g.diff"
   }
 }