New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 838990 link

Starred by 2 users

Issue metadata

Status: WontFix
Owner: ----
Closed: Jan 3
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocking:
issue 759794



Sign in to add a comment

Try link in parallel on the bot

Project Member Reported by mmoroz@chromium.org, May 2 2018

Issue description

Current implementation builds targets consequently, i.e. one at a time. A more efficient option would be to pass all the targets to ninja at once, so that parallel linking can be done. However, there are following potential issues:

1) link pool limits may need to be adjusted: https://cs.chromium.org/chromium/src/build/toolchain/concurrent_links.gni?sq=package:chromium&dr=C&l=28

2) it's going to be harder for debugging with current implementation in bash scripts (https://chrome-internal.googlesource.com/chrome/tools/code-coverage/+/master/scripts/)

3) we'll eventually face this issue once we start building using Chrome Infra recipes, that's when we'll take a closer look


 
Components: Tools>CodeCoverage
Status: WontFix (was: Untriaged)
I guess this is WontFix, as the currents bots are working OK for now, and in Chrome Infra implementation that should be already taken care of.

Sign in to add a comment