[PATCH] D124715: Use QoS class Utility for ThreadPriority::Low on Mac
Argyrios Kyrtzidis via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed May 4 11:12:32 PDT 2022
akyrtzi added a comment.
My recommendation is that indexing belongs to 'utility', as @stefanhaller mentioned the user is actively depending on functionality coming from the index.
That said, you may want to consider dynamically switching to background if running on laptop with battery, or other heuristics, but that could be a follow-up enhancement, I don't think always using 'background' is appropriate.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D124715/new/
https://reviews.llvm.org/D124715
More information about the cfe-commits
mailing list