[llvm] [readtapi] Cleanup usages of ExitOnError, NFCI (PR #76745)
Cyndy Ishida via llvm-commits
llvm-commits at lists.llvm.org
Tue Jan 2 11:20:39 PST 2024
https://github.com/cyndyishida created https://github.com/llvm/llvm-project/pull/76745
None
>From 4dd8b9dc99ea9dbcb0ed05fa9ef5ca84d63bfd9d Mon Sep 17 00:00:00 2001
From: Cyndy Ishida <cyndy_ishida at apple.com>
Date: Tue, 2 Jan 2024 11:01:33 -0800
Subject: [PATCH] [readtapi] Cleanup usages of ExitOnError, NFCI
---
llvm/tools/llvm-readtapi/llvm-readtapi.cpp | 21 +++++----------------
1 file changed, 5 insertions(+), 16 deletions(-)
diff --git a/llvm/tools/llvm-readtapi/llvm-readtapi.cpp b/llvm/tools/llvm-readtapi/llvm-readtapi.cpp
index 193a281d6341af..4bd47223ea91f5 100644
--- a/llvm/tools/llvm-readtapi/llvm-readtapi.cpp
+++ b/llvm/tools/llvm-readtapi/llvm-readtapi.cpp
@@ -108,20 +108,12 @@ getInterfaceFile(const StringRef Filename, bool ResetBanner = true) {
LLVM_FALLTHROUGH;
case file_magic::macho_dynamically_linked_shared_lib_stub:
LLVM_FALLTHROUGH;
- case file_magic::macho_universal_binary: {
- auto IFOrErr = DylibReader::get(Buffer->getMemBufferRef());
- if (!IFOrErr)
- ExitOnErr(IFOrErr.takeError());
- IF = std::move(*IFOrErr);
+ case file_magic::macho_universal_binary:
+ IF = ExitOnErr(DylibReader::get(Buffer->getMemBufferRef()));
break;
- }
- case file_magic::tapi_file: {
- auto IFOrErr = TextAPIReader::get(Buffer->getMemBufferRef());
- if (!IFOrErr)
- ExitOnErr(IFOrErr.takeError());
- IF = std::move(*IFOrErr);
+ case file_magic::tapi_file:
+ IF = ExitOnErr(TextAPIReader::get(Buffer->getMemBufferRef()));
break;
- }
default:
reportError(Filename + ": unsupported file type");
}
@@ -170,10 +162,7 @@ static bool handleMergeAction(const Context &Ctx) {
Out = std::move(IF);
continue;
}
- auto ResultIF = Out->merge(IF.get());
- if (!ResultIF)
- ExitOnErr(ResultIF.takeError());
- Out = std::move(ResultIF.get());
+ Out = ExitOnErr(Out->merge(IF.get()));
}
return handleWriteAction(Ctx, std::move(Out));
}
More information about the llvm-commits
mailing list