[clang] [clang analyzer]consume `llvm::Error` (PR #120597)
via cfe-commits
cfe-commits at lists.llvm.org
Thu Dec 19 08:11:05 PST 2024
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-clang
Author: Congcong Cai (HerrCai0907)
<details>
<summary>Changes</summary>
`llvm::Error` must be consumed, otherwise it will cause trap during destructor
---
Full diff: https://github.com/llvm/llvm-project/pull/120597.diff
1 Files Affected:
- (modified) clang/lib/StaticAnalyzer/Core/TextDiagnostics.cpp (+1-1)
``````````diff
diff --git a/clang/lib/StaticAnalyzer/Core/TextDiagnostics.cpp b/clang/lib/StaticAnalyzer/Core/TextDiagnostics.cpp
index 71268af22e2424..e8cf367b833467 100644
--- a/clang/lib/StaticAnalyzer/Core/TextDiagnostics.cpp
+++ b/clang/lib/StaticAnalyzer/Core/TextDiagnostics.cpp
@@ -81,7 +81,7 @@ class TextDiagnostics : public PathDiagnosticConsumer {
if (llvm::Error Err = Repls.add(Repl)) {
llvm::errs() << "Error applying replacement " << Repl.toString()
- << ": " << Err << "\n";
+ << ": " << llvm::toString(std::move(Err)) << "\n";
}
}
};
``````````
</details>
https://github.com/llvm/llvm-project/pull/120597
More information about the cfe-commits
mailing list