Changeset 222442 in webkit


Ignore:
Timestamp:
Sep 25, 2017 5:21:47 AM (7 years ago)
Author:
clopez@igalia.com
Message:

REGRESSION(r222160) [GTK] [Debug] Internal compiler error on the buildbot (huge memory usage by GCC)
https://bugs.webkit.org/show_bug.cgi?id=177223

Reviewed by Carlos Garcia Campos.

When the user environment sets the variable NUMBER_OF_PROCESSORS
this should be passed down to the build system (ninja or make) always.

Playing with the number of this variable is needed when using icecc
(as you usually want to launch more parallel build process than your
number of cores), or when you want to limit the number of parallel
build process for hardware capacity reasons.

  • Scripts/build-webkit:
Location:
trunk/Tools
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/Tools/ChangeLog

    r222439 r222442  
     12017-09-25  Carlos Alberto Lopez Perez  <clopez@igalia.com>
     2
     3        REGRESSION(r222160) [GTK] [Debug] Internal compiler error on the buildbot (huge memory usage by GCC)
     4        https://bugs.webkit.org/show_bug.cgi?id=177223
     5
     6        Reviewed by Carlos Garcia Campos.
     7
     8        When the user environment sets the variable NUMBER_OF_PROCESSORS
     9        this should be passed down to the build system (ninja or make) always.
     10
     11        Playing with the number of this variable is needed when using icecc
     12        (as you usually want to launch more parallel build process than your
     13        number of cores), or when you want to limit the number of parallel
     14        build process for hardware capacity reasons.
     15
     16        * Scripts/build-webkit:
     17
    1182017-09-24  Fujii Hironori  <Hironori.Fujii@sony.com>
    219
  • trunk/Tools/Scripts/build-webkit

    r222400 r222442  
    240240
    241241if (isCMakeBuild() && !isAnyWindows()) {
    242     if (!canUseNinja()) {
    243         # By default we build using all of the available CPUs.
    244         # Ninja will automatically determine the number of jobs to run in parallel,
    245         # so don't override the number of jobs if building with Ninja.
     242    if (!canUseNinja() || defined($ENV{NUMBER_OF_PROCESSORS})) {
     243        # If the user environment is not setting a specific number of process,
     244        # then don't pass the number of jobs to Ninja. Because Ninja will
     245        # automatically determine the number of jobs to run in parallel.
    246246        $makeArgs .= ($makeArgs ? " " : "") . "-j" . numberOfCPUs() if $makeArgs !~ /-j\s*\d+/;
    247247    }
Note: See TracChangeset for help on using the changeset viewer.