[lld] [LLD, MachO] Default `objc_relative_method_lists` on MacOS11+/iOS14+ (PR #101246)

via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 30 14:42:40 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 bb4aeb66ab4f39901ef6c1d01baf8b82b81cecfb eed10155dea1d65414c191a8a8b75ff64686a2b3 --extensions cpp -- lld/MachO/Driver.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 c893841b4e..bcb610929e 100644
--- a/lld/MachO/Driver.cpp
+++ b/lld/MachO/Driver.cpp
@@ -1145,13 +1145,13 @@ static bool shouldEmitRelativeMethodLists(const InputArgList &args) {
   //   - default false on everything else
   PlatformType platform = config->platformInfo.target.Platform;
   switch (platform) {
-    case PLATFORM_IOS:
-    case PLATFORM_IOSSIMULATOR:
-      return config->platformInfo.target.MinDeployment >= VersionTuple(14, 0);
-    case PLATFORM_MACOS:
-      return config->platformInfo.target.MinDeployment >= VersionTuple(11, 0);
-    default:
-      return false;
+  case PLATFORM_IOS:
+  case PLATFORM_IOSSIMULATOR:
+    return config->platformInfo.target.MinDeployment >= VersionTuple(14, 0);
+  case PLATFORM_MACOS:
+    return config->platformInfo.target.MinDeployment >= VersionTuple(11, 0);
+  default:
+    return false;
   };
   llvm_unreachable("Shouldn't be here");
 }

``````````

</details>


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


More information about the llvm-commits mailing list