tree babd158db382b570a612672406fe87779066c577
parent c91f7e86727683118b6096757717135463fdcdd7
author Takuto Ikuta <tikuta@google.com> 1774339313 +0900
committer GN LUCI <gn-scoped@luci-project-accounts.iam.gserviceaccount.com> 1774350513 -0700

Limit the maximum number of GN worker threads to 32.

On machines with high core counts (e.g., 128 cores), using all available
CPUs for worker threads significantly slows down `gn gen` due to thread
scheduling overhead and contention.

This change caps the default number of worker threads at 32 while
maintaining the existing minimum of 8.

Profiling results of `gn gen` for Chrome using hyperfine on a 128-core
Linux machine:

Summary
  ./buildtools/linux64/gn gen out/Default/ --threads=32 ran
    1.01 ± 0.03 times faster than ./buildtools/linux64/gn gen out/Default/ --threads=16
    1.07 ± 0.03 times faster than ./buildtools/linux64/gn gen out/Default/ --threads=48
    1.13 ± 0.03 times faster than ./buildtools/linux64/gn gen out/Default/ --threads=64
    1.26 ± 0.03 times faster than ./buildtools/linux64/gn gen out/Default/ --threads=96
    1.28 ± 0.04 times faster than ./buildtools/linux64/gn gen out/Default/ --threads=8

This improved gn gen for android too.

Summary
  ./buildtools/linux64/gn --threads=32 gen out/android ran
    1.19 ± 0.04 times faster than ./buildtools/linux64/gn gen out/android

Bug: 484863025
Change-Id: I44b467c1813a32015666926d8bbaaad4206b5447
Reviewed-on: https://gn-review.googlesource.com/c/gn/+/21740
Reviewed-by: David Turner <digit@google.com>
Commit-Queue: Takuto Ikuta <tikuta@google.com>