[Lldb-commits] [lldb] [lldb] Include `<chrono>` for `system_clock` (PR #118059)

via lldb-commits lldb-commits at lists.llvm.org
Thu Nov 28 23:37:51 PST 2024


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-lldb

Author: Lily Wang (LilyWangLL)

<details>
<summary>Changes</summary>

I am a member of Microsoft vcpkg, due to there are new changes merged by microsoft/STL#<!-- -->5105, which revealed a conformance issue in `llvm`. It must add include `<chrono>` to fix this error.

Compiler error with this STL change:
```
D:\b\llvm\src\org-18.1.6-e754cb1d0b.clean\lldb\tools\lldb-dap\ProgressEvent.h(79): error C2039: 'system_clock': is not a member of 'std::chrono'
D:\b\llvm\src\org-18.1.6-e754cb1d0b.clean\lldb\tools\lldb-dap\ProgressEvent.cpp(134): error C2039: 'now': is not a member of 'std::chrono'
```


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


1 Files Affected:

- (modified) lldb/tools/lldb-dap/ProgressEvent.h (+1) 


``````````diff
diff --git a/lldb/tools/lldb-dap/ProgressEvent.h b/lldb/tools/lldb-dap/ProgressEvent.h
index dac21977add2d0..72317b879c803a 100644
--- a/lldb/tools/lldb-dap/ProgressEvent.h
+++ b/lldb/tools/lldb-dap/ProgressEvent.h
@@ -7,6 +7,7 @@
 //===----------------------------------------------------------------------===//
 
 #include <atomic>
+#include <chrono>
 #include <mutex>
 #include <optional>
 #include <queue>

``````````

</details>


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


More information about the lldb-commits mailing list