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

Issue 838340 link

Starred by 2 users

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 3
Type: Bug



Sign in to add a comment

gn doesn't use `deps = gcc` for depfiles on actions

Project Member Reported by thakis@chromium.org, Apr 30 2018

Issue description

Ninja has a deps cache where it reads .d files right after the command producing it has completed and stores the depfile contents in .ninja_deps in a fairly efficient binary format in the build dir. That way, ninja doesn't have to parse .d files from the disk at startup.

The deps cache is opt in. gn doesn't use it.

For a build of chrome on linux, only 182 depfiles get generated, and `-d stats` says parsing them only takes 2ms, so that's not a huge thing.

But on Android, out/gnand/toolchain.ninja contains 6502 depfiles, it might help more there. (But assuming the 2ms scale up linearly, still just 0.1s or so, so still not a huge win. I haven't tried `-d stats` on an android build.)

Might be nice to do for cleanliness still.
 
Cc: tikuta@chromium.org

Comment 2 by wnwen@chromium.org, May 7 2018

Cc: wnwen@chromium.org

Comment 3 by wnwen@chromium.org, May 7 2018

Components: Build
Status: Available (was: Untriaged)

Sign in to add a comment