This is an archive of the discontinued LLVM Phabricator instance.

[clangd] Reduce number of threads used by BackgroundIndex to number of physical cores.
ClosedPublic

Authored by kadircet on Feb 6 2019, 7:12 AM.

Diff Detail

Event Timeline

kadircet created this revision.Feb 6 2019, 7:12 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 6 2019, 7:12 AM
ilya-biryukov accepted this revision.Feb 7 2019, 7:31 AM

LGTM.
NIT about a description: we can't be 100% certain that it's related to hyper-threading, so I'd avoid putting that to the description, maybe simply mention that this avoids higher latency for foreground operations?

This revision is now accepted and ready to land.Feb 7 2019, 7:31 AM
kadircet edited the summary of this revision. (Show Details)Feb 7 2019, 8:02 AM
This revision was automatically updated to reflect the committed changes.