This revision was automatically updated to reflect the committed changes.
Closed by commit rCTE353422: [clangd] Reduce number of threads used by
BackgroundIndex to number of physical… (authored by kadircet, committed by ).
Changed prior to commit:
https://reviews.llvm.org/D57819?vs=185549&id=18
ilya-biryukov accepted this revision.
ilya-biryukov added a comment.
This revision is now accepted and ready to land.
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
kadircet created this revision.
kadircet added a reviewer: ilya-biryukov.
Herald added subscribers: cfe-commits, arphaman, jkorous, MaskRay, ioeric.
Herald added a project: clang.
clangd is using as many threads as logical cores for BackgroundIndex
by default. Due to hyper-threading this causes ca