[clang-tools-extra] [clangd][trace] Fix comment to mention that trace spans are measured … (PR #86938)

via cfe-commits cfe-commits at lists.llvm.org
Thu Mar 28 04:47:53 PDT 2024


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-clang-tools-extra

@llvm/pr-subscribers-clangd

Author: None (VitaNuo)

<details>
<summary>Changes</summary>

…in milliseconds rather than seconds.

---
Full diff: https://github.com/llvm/llvm-project/pull/86938.diff


1 Files Affected:

- (modified) clang-tools-extra/clangd/support/Trace.h (+2-2) 


``````````diff
diff --git a/clang-tools-extra/clangd/support/Trace.h b/clang-tools-extra/clangd/support/Trace.h
index 1bfc75b874d8a9..36c3745a41e969 100644
--- a/clang-tools-extra/clangd/support/Trace.h
+++ b/clang-tools-extra/clangd/support/Trace.h
@@ -143,8 +143,8 @@ bool enabled();
 class Span {
 public:
   Span(llvm::Twine Name);
-  /// Records span's duration in seconds to \p LatencyMetric with \p Name as the
-  /// label.
+  /// Records span's duration in milliseconds to \p LatencyMetric with \p Name
+  /// as the label.
   Span(llvm::Twine Name, const Metric &LatencyMetric);
   ~Span();
 

``````````

</details>


https://github.com/llvm/llvm-project/pull/86938


More information about the cfe-commits mailing list