[lld] [lld-macho] Fix erasing category names for ObjC categories (PR #98417)

via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 10 16:52:57 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 ceaaa19238ac9ca3e42503eca4d4d2215b27bbc1 625ee36bcf930eb7f745806a48d8c1fc141f3359 --extensions cpp -- lld/MachO/ObjC.cpp
``````````

</details>

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

``````````diff
diff --git a/lld/MachO/ObjC.cpp b/lld/MachO/ObjC.cpp
index 4c08289b2e..c0216117ef 100644
--- a/lld/MachO/ObjC.cpp
+++ b/lld/MachO/ObjC.cpp
@@ -1309,12 +1309,12 @@ void ObjcCategoryMerger::eraseMergedCategories() {
 
       eraseISec(catInfo.catBodyIsec);
 
-      // We can't erase 'catLayout.nameOffset' for either Swift or ObjC 
-      //   categories because the name will sometimes also be used for other 
+      // We can't erase 'catLayout.nameOffset' for either Swift or ObjC
+      //   categories because the name will sometimes also be used for other
       //   purposes.
-      // For Swift, see  usages of 'l_.str.11.SimpleClass' in 
+      // For Swift, see  usages of 'l_.str.11.SimpleClass' in
       //   objc-category-merging-swift.s
-      // For ObjC, see usages of 'l_OBJC_CLASS_NAME_.1' in 
+      // For ObjC, see usages of 'l_OBJC_CLASS_NAME_.1' in
       //   objc-category-merging-erase-objc-name-test.s
       // TODO: handle the above in a smarter way
 

``````````

</details>


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


More information about the llvm-commits mailing list