From b3c8554f28a95063e406924c25336e0da7efb4fd Mon Sep 17 00:00:00 2001 From: Elliot Goodrich Date: Sun, 25 Jun 2023 16:09:36 +0100 Subject: [PATCH] Revert "[llvm] Move StringExtras.h include from Error.h to Error.cpp" This reverts commit 2fa0dbd7bf353db98b27b56137cced0a67de2748. --- llvm/include/llvm/Support/Error.h | 10 +++++++++- llvm/lib/Support/Error.cpp | 11 ----------- 2 files changed, 9 insertions(+), 12 deletions(-) diff --git a/llvm/include/llvm/Support/Error.h b/llvm/include/llvm/Support/Error.h index 2292770..0f1b632 100644 --- a/llvm/include/llvm/Support/Error.h +++ b/llvm/include/llvm/Support/Error.h @@ -14,6 +14,8 @@ #define LLVM_SUPPORT_ERROR_H #include "llvm-c/Error.h" +#include "llvm/ADT/SmallVector.h" +#include "llvm/ADT/StringExtras.h" #include "llvm/ADT/Twine.h" #include "llvm/Config/abi-breaking.h" #include "llvm/Support/AlignOf.h" @@ -1023,7 +1025,13 @@ void logAllUnhandledErrors(Error E, raw_ostream &OS, Twine ErrorBanner = {}); /// Write all error messages (if any) in E to a string. The newline character /// is used to separate error messages. -std::string toString(Error E); +inline std::string toString(Error E) { + SmallVector Errors; + handleAllErrors(std::move(E), [&Errors](const ErrorInfoBase &EI) { + Errors.push_back(EI.message()); + }); + return join(Errors.begin(), Errors.end(), "\n"); +} /// Consume a Error without doing anything. This method should be used /// only where an error can be considered a reasonable and expected return diff --git a/llvm/lib/Support/Error.cpp b/llvm/lib/Support/Error.cpp index 21d5915..b339b70 100644 --- a/llvm/lib/Support/Error.cpp +++ b/llvm/lib/Support/Error.cpp @@ -7,8 +7,6 @@ //===----------------------------------------------------------------------===// #include "llvm/Support/Error.h" -#include "llvm/ADT/SmallVector.h" -#include "llvm/ADT/StringExtras.h" #include "llvm/ADT/Twine.h" #include "llvm/Support/ErrorHandling.h" #include @@ -72,15 +70,6 @@ void logAllUnhandledErrors(Error E, raw_ostream &OS, Twine ErrorBanner) { }); } -/// Write all error messages (if any) in E to a string. The newline character -/// is used to separate error messages. -std::string toString(Error E) { - SmallVector Errors; - handleAllErrors(std::move(E), [&Errors](const ErrorInfoBase &EI) { - Errors.push_back(EI.message()); - }); - return join(Errors.begin(), Errors.end(), "\n"); -} std::error_code ErrorList::convertToErrorCode() const { return std::error_code(static_cast(ErrorErrorCode::MultipleErrors), -- 2.7.4