[PATCH] D57819: [clangd] Reduce number of threads used by BackgroundIndex to number of physical cores.

2019-02-07 Thread Kadir Cetinkaya via Phabricator via cfe-commits
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

[PATCH] D57819: [clangd] Reduce number of threads used by BackgroundIndex to number of physical cores.

2019-02-07 Thread Ilya Biryukov via Phabricator via cfe-commits
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

[PATCH] D57819: [clangd] Reduce number of threads used by BackgroundIndex to number of physical cores.

2019-02-06 Thread Kadir Cetinkaya via Phabricator via cfe-commits
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