New issue
Advanced search Search tips

Issue 773144 link

Starred by 4 users

Issue metadata

Status: Closed
Owner: ----
Closed: Nov 12
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 3
Type: Bug

Blocking:
issue 787983



Sign in to add a comment

Get v8_context_snapshot_generator out of critical build path

Project Member Reported by brucedaw...@chromium.org, Oct 10 2017

Issue description

On incremental non-component release builds the linking of v8_context_snapshot_generator.exe can take 90+ s. It blocks the linking of chrome_child.dll which ends up being the long pole which means that the linking of v8_context_snapshot_generator.exe ends up slowing down the build by 90+ s, which often represents 25% of the total build time.

I tried disabling it by setting use_v8_context_snapshot to false but that doesn't work.
I tried turning of linking with symbols but that only helped a tiny amount.
I don't actually have any suggestions, just the observation that this makes non-component builds even more painful.

 
IncrementalReleaseBuild.PNG
15.7 KB View Download
Blocking: 495670
Blocking: 787983
Blocking: -495670

Comment 4 by tikuta@chromium.org, Dec 14 2017

Cc: peria@chromium.org
+peria@, CC'd owner of v8_context_snapshot_generator

Comment 5 by tikuta@chromium.org, Dec 20 2017

Cc: tikuta@chromium.org
Owner: tikuta@chromium.org
Status: Assigned (was: Untriaged)
Ikuta-san, wasn't this issue fixed?
Labels: OS-Windows
Owner: brucedaw...@chromium.org
brucedawson@, is this still problematic?

I think this is not so problematic after migration to lld.
Owner: ----
Status: Closed (was: Assigned)
In my build in the same config with win7_chromium_rel_ng builder, linking time of v8_context_snapshot_generator is around 6 seconds.
So I think we can close this now.

brucedawson@, if you see slowness yet, please re-open this issue.

Sign in to add a comment