Skip to content

Conversation

@kosarev
Copy link
Collaborator

@kosarev kosarev commented Nov 4, 2024

No description provided.

@kosarev kosarev requested a review from MaskRay November 4, 2024 14:11
@llvmbot llvmbot added the llvm:mc Machine (object) code label Nov 4, 2024
@llvmbot
Copy link
Member

llvmbot commented Nov 4, 2024

@llvm/pr-subscribers-mc

Author: Ivan Kosarev (kosarev)

Changes

Full diff: https://github.com/llvm/llvm-project/pull/114798.diff

1 Files Affected:

  • (modified) llvm/lib/MC/MCContext.cpp (-3)
diff --git a/llvm/lib/MC/MCContext.cpp b/llvm/lib/MC/MCContext.cpp
index b97f9d9f5fed0f..a4404b5d1f8d4f 100644
--- a/llvm/lib/MC/MCContext.cpp
+++ b/llvm/lib/MC/MCContext.cpp
@@ -575,9 +575,6 @@ MCSectionELF *MCContext::getELFSection(const Twine &Section, unsigned Type,
                                        const MCSymbolELF *GroupSym,
                                        bool IsComdat, unsigned UniqueID,
                                        const MCSymbolELF *LinkedToSym) {
-  StringRef Group = "";
-  if (GroupSym)
-    Group = GroupSym->getName();
   assert(!(LinkedToSym && LinkedToSym->getName().empty()));
 
   // Sections are differentiated by the quadruple (section_name, group_name,

@MaskRay MaskRay merged commit bcf8ded into llvm:main Jun 6, 2025
5 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

llvm:mc Machine (object) code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants