[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