[lld] [lld-macho] Save all thin archive members in repro tarball (PR #97169)

via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 18 02:10:32 PDT 2024


github-actions[bot] wrote:

<!--LLVM CODE FORMAT COMMENT: {clang-format}-->


:warning: C/C++ code formatter, clang-format found issues in your code. :warning:

<details>
<summary>
You can test this locally with the following command:
</summary>

``````````bash
git-clang-format --diff b1864a8d6ab8bfd346922e36d80e684a4eaf3248 851013ed1b8729b50fa103272f95aa0c5236567d --extensions h,cpp -- lld/MachO/Config.h lld/MachO/Driver.cpp lld/MachO/InputFiles.cpp
``````````

</details>

<details>
<summary>
View the diff from clang-format here.
</summary>

``````````diff
diff --git a/lld/MachO/Driver.cpp b/lld/MachO/Driver.cpp
index 2b881e8010..dc9d635b48 100644
--- a/lld/MachO/Driver.cpp
+++ b/lld/MachO/Driver.cpp
@@ -347,14 +347,13 @@ static InputFile *addFile(StringRef path, LoadType loadType,
               reason = "-all_load";
               break;
           }
-            if (Error e = file->fetch(c, reason)) {
-              if (config->warnThinArchiveMissingMembers)
-                warn(toString(file) + ": " + reason +
-                     " failed to load archive member: " +
-                     toString(std::move(e)));
-              else
-                llvm::consumeError(std::move(e));
-            }
+          if (Error e = file->fetch(c, reason)) {
+            if (config->warnThinArchiveMissingMembers)
+              warn(toString(file) + ": " + reason +
+                   " failed to load archive member: " + toString(std::move(e)));
+            else
+              llvm::consumeError(std::move(e));
+          }
         }
         if (e)
           error(toString(file) +

``````````

</details>


https://github.com/llvm/llvm-project/pull/97169


More information about the llvm-commits mailing list