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

Issue 11026061: Parallelize ninja generation when GYP_PARALLEL=1. (Closed)

Created:
8 years, 2 months ago by dmazzoni
Modified:
8 years, 2 months ago
Reviewers:
Nico, scottmg
CC:
gyp-developer_googlegroups.com
Base URL:
http://git.chromium.org/external/gyp.git@master
Visibility:
Public.

Description

Parallelize ninja generation when GYP_PARALLEL=1. This is a really easy approach, parallelizing by configs (Debug/Release). It's probably not worth parallelizing this step much more because there's some overhead and the total time of this step isn't that large.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+20 lines, -8 lines) Patch
M pylib/gyp/__init__.py View 3 chunks +5 lines, -5 lines 0 comments Download
M pylib/gyp/generator/ninja.py View 3 chunks +15 lines, -3 lines 0 comments Download

Messages

Total messages: 11 (0 generated)
dmazzoni
This should shave a few more seconds off the total time.
8 years, 2 months ago (2012-10-05 06:42:47 UTC) #1
Nico
Wait, isn't GYP_PARALLEL implemented already? What does it currently parallelize? How much does this change ...
8 years, 2 months ago (2012-10-05 06:45:25 UTC) #2
Ami GONE FROM CHROMIUM
drive-by FWIW, this patch doesn't make a difference on my z620/gprecise/ninja setup. Before CL: fischman@fischman-linux ...
8 years, 2 months ago (2012-10-05 06:52:18 UTC) #3
dmazzoni
On 2012/10/05 06:45:25, Nico wrote: > Wait, isn't GYP_PARALLEL implemented already? What does it currently ...
8 years, 2 months ago (2012-10-05 07:06:49 UTC) #4
Nico
Ok. Any reason to not just always do this, instead of having it behind a ...
8 years, 2 months ago (2012-10-05 07:09:34 UTC) #5
dmazzoni
On 2012/10/05 07:09:34, Nico wrote: > Ok. > > Any reason to not just always ...
8 years, 2 months ago (2012-10-05 07:14:17 UTC) #6
Nico
On 2012/10/05 07:14:17, Dominic Mazzoni wrote: > On 2012/10/05 07:09:34, Nico wrote: > > Ok. ...
8 years, 2 months ago (2012-10-05 08:14:09 UTC) #7
scottmg
Super, thanks! (this was at my behest because windows spends about 4x the time in ...
8 years, 2 months ago (2012-10-05 16:29:43 UTC) #8
dmazzoni
On 2012/10/05 16:29:43, scottmg wrote: > Non-parallel: 57.8s > Loading-only parallel: 46.0s > Loading and ...
8 years, 2 months ago (2012-10-08 17:02:46 UTC) #9
scottmg
On 2012/10/08 17:02:46, Dominic Mazzoni wrote: > On 2012/10/05 16:29:43, scottmg wrote: > > Non-parallel: ...
8 years, 2 months ago (2012-10-08 17:11:22 UTC) #10
Nico
8 years, 2 months ago (2012-10-14 01:13:56 UTC) #11
lgtm

Maybe you can add a TODO with a link to the bug for removing the option

Powered by Google App Engine
This is Rietveld 408576698