[clang-tools-extra] 1aa3a54 - [clangd] Dont include version string in update tasks

Kadir Cetinkaya via cfe-commits cfe-commits at lists.llvm.org
Tue Apr 19 10:27:31 PDT 2022


Author: Kadir Cetinkaya
Date: 2022-04-19T19:27:04+02:00
New Revision: 1aa3a54921a1e6bb43ff14c072b9129aa78d6988

URL: https://github.com/llvm/llvm-project/commit/1aa3a54921a1e6bb43ff14c072b9129aa78d6988
DIFF: https://github.com/llvm/llvm-project/commit/1aa3a54921a1e6bb43ff14c072b9129aa78d6988.diff

LOG: [clangd] Dont include version string in update tasks

This increases cardinality of span latency metrics. Currently this was
being shown to the user via file status updates as `Running Update (x)` after
this change we'll only display `Running Update`. This also affects logs in case
of a crash, but contents and version number for inputs are printed separately in
that case already.

Differential Revision: https://reviews.llvm.org/D124013

Added: 
    

Modified: 
    clang-tools-extra/clangd/TUScheduler.cpp

Removed: 
    


################################################################################
diff  --git a/clang-tools-extra/clangd/TUScheduler.cpp b/clang-tools-extra/clangd/TUScheduler.cpp
index 7c5a52f6ea443..19d4ca5a48a50 100644
--- a/clang-tools-extra/clangd/TUScheduler.cpp
+++ b/clang-tools-extra/clangd/TUScheduler.cpp
@@ -798,7 +798,7 @@ ASTWorker::~ASTWorker() {
 
 void ASTWorker::update(ParseInputs Inputs, WantDiagnostics WantDiags,
                        bool ContentChanged) {
-  std::string TaskName = llvm::formatv("Update ({0})", Inputs.Version);
+  llvm::StringLiteral TaskName = "Update";
   auto Task = [=]() mutable {
     // Get the actual command as `Inputs` does not have a command.
     // FIXME: some build systems like Bazel will take time to preparing


        


More information about the cfe-commits mailing list