[clang] 3f1267e - [NFC][clang-offload-bundler] Simplify main function (#138555)

via cfe-commits cfe-commits at lists.llvm.org
Mon May 5 20:01:45 PDT 2025


Author: Alexey Bader
Date: 2025-05-05T20:01:42-07:00
New Revision: 3f1267e1b39c31cea390c7425e88b01d3304d8c7

URL: https://github.com/llvm/llvm-project/commit/3f1267e1b39c31cea390c7425e88b01d3304d8c7
DIFF: https://github.com/llvm/llvm-project/commit/3f1267e1b39c31cea390c7425e88b01d3304d8c7.diff

LOG: [NFC][clang-offload-bundler] Simplify main function (#138555)

Applied "no else after return" rule from the LLVM's Coding Standards
https://llvm.org/docs/CodingStandards.html#don-t-use-else-after-a-return

Added: 
    

Modified: 
    clang/tools/clang-offload-bundler/ClangOffloadBundler.cpp

Removed: 
    


################################################################################
diff  --git a/clang/tools/clang-offload-bundler/ClangOffloadBundler.cpp b/clang/tools/clang-offload-bundler/ClangOffloadBundler.cpp
index 49412aeeba7fc..75de97068e6e9 100644
--- a/clang/tools/clang-offload-bundler/ClangOffloadBundler.cpp
+++ b/clang/tools/clang-offload-bundler/ClangOffloadBundler.cpp
@@ -426,12 +426,9 @@ int main(int argc, const char **argv) {
   OffloadBundler Bundler(BundlerConfig);
 
   return doWork([&]() {
-    if (Unbundle) {
-      if (BundlerConfig.FilesType == "a")
-        return Bundler.UnbundleArchive();
-      else
-        return Bundler.UnbundleFiles();
-    } else
-      return Bundler.BundleFiles();
+    if (Unbundle)
+      return (BundlerConfig.FilesType == "a") ? Bundler.UnbundleArchive()
+                                              : Bundler.UnbundleFiles();
+    return Bundler.BundleFiles();
   });
 }


        


More information about the cfe-commits mailing list