[llvm] [NFC][llvm-objcopy] Eliminate else after return in llvm-objcopy (PR #177112)
via llvm-commits
llvm-commits at lists.llvm.org
Wed Jan 21 00:05:57 PST 2026
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-llvm-binary-utilities
Author: Ruoyu Qiu (cabbaken)
<details>
<summary>Changes</summary>
Eliminate else after return in `getDriverConfig`.
---
Full diff: https://github.com/llvm/llvm-project/pull/177112.diff
1 Files Affected:
- (modified) llvm/tools/llvm-objcopy/llvm-objcopy.cpp (+6-4)
``````````diff
diff --git a/llvm/tools/llvm-objcopy/llvm-objcopy.cpp b/llvm/tools/llvm-objcopy/llvm-objcopy.cpp
index ad67b673b2cc7..b907e023da31e 100644
--- a/llvm/tools/llvm-objcopy/llvm-objcopy.cpp
+++ b/llvm/tools/llvm-objcopy/llvm-objcopy.cpp
@@ -73,12 +73,14 @@ static Expected<DriverConfig> getDriverConfig(ArrayRef<const char *> Args) {
if (Is("bitcode-strip") || Is("bitcode_strip"))
return parseBitcodeStripOptions(Args, reportWarning);
- else if (Is("strip"))
+
+ if (Is("strip"))
return parseStripOptions(Args, reportWarning);
- else if (Is("install-name-tool") || Is("install_name_tool"))
+
+ if (Is("install-name-tool") || Is("install_name_tool"))
return parseInstallNameToolOptions(Args);
- else
- return parseObjcopyOptions(Args, reportWarning);
+
+ return parseObjcopyOptions(Args, reportWarning);
}
/// The function executeObjcopyOnIHex does the dispatch based on the format
``````````
</details>
https://github.com/llvm/llvm-project/pull/177112
More information about the llvm-commits
mailing list