[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