[llvm] [AsmPrinter] Link .section_sizes to the correct section (PR #135583)
Sergei Barannikov via llvm-commits
llvm-commits at lists.llvm.org
Sun Apr 13 20:22:02 PDT 2025
https://github.com/s-barannikov created https://github.com/llvm/llvm-project/pull/135583
AsmPrinter may switch the current section when e.g., emitting a jump table for a switch. `.stack_sizes` should still be linked to the function section. If the section is wrong, objdump emits a warning "relocation symbol is not in the expected section".
>From 3ceba6145b27575cde0ad5b2d5a01ab90671c645 Mon Sep 17 00:00:00 2001
From: Sergei Barannikov <barannikov88 at gmail.com>
Date: Mon, 14 Apr 2025 05:59:57 +0300
Subject: [PATCH] [AsmPrinter] Link .section_sizes to the correct section
AsmPrinter may switch the current section when e.g., emitting a jump
table for a switch. `.stack_sizes` should still be linked to the
function section. If the section is wrong, objdump emits a warning
"relocation symbol is not in the expected section".
---
llvm/lib/CodeGen/AsmPrinter/AsmPrinter.cpp | 2 +-
.../CodeGen/SystemZ/stack-size-section.ll | 30 +++++++++++++++++++
2 files changed, 31 insertions(+), 1 deletion(-)
diff --git a/llvm/lib/CodeGen/AsmPrinter/AsmPrinter.cpp b/llvm/lib/CodeGen/AsmPrinter/AsmPrinter.cpp
index cf8f1c878ea5a..821879a1f7c36 100644
--- a/llvm/lib/CodeGen/AsmPrinter/AsmPrinter.cpp
+++ b/llvm/lib/CodeGen/AsmPrinter/AsmPrinter.cpp
@@ -1586,7 +1586,7 @@ void AsmPrinter::emitStackSizeSection(const MachineFunction &MF) {
return;
MCSection *StackSizeSection =
- getObjFileLowering().getStackSizesSection(*getCurrentSection());
+ getObjFileLowering().getStackSizesSection(*MF.getSection());
if (!StackSizeSection)
return;
diff --git a/llvm/test/CodeGen/SystemZ/stack-size-section.ll b/llvm/test/CodeGen/SystemZ/stack-size-section.ll
index 024f20bfc2dc8..6225ce0ef23a1 100644
--- a/llvm/test/CodeGen/SystemZ/stack-size-section.ll
+++ b/llvm/test/CodeGen/SystemZ/stack-size-section.ll
@@ -38,4 +38,34 @@ define void @dynalloc(i32 %N) #0 {
ret void
}
+; Check that .stack_sizes section is linked to the function's section (.text),
+; and not to the section containing the jump table (.rodata).
+; CHECK-LABEL: .section .stack_sizes,"o", at progbits,.text{{$}}
+; CHECK-LABEL: .quad .Lfunc_begin4
+; CHECK-LABEL: .ascii "\260!"
+define i32 @jump_table(i32 %x) {
+ %arr = alloca [1024 x i32]
+ switch i32 %x, label %sw.epilog [
+ i32 0, label %sw.bb0
+ i32 1, label %sw.bb1
+ i32 2, label %sw.bb2
+ i32 3, label %sw.bb3
+ ]
+
+sw.bb0:
+ ret i32 0
+
+sw.bb1:
+ ret i32 1
+
+sw.bb2:
+ ret i32 2
+
+sw.bb3:
+ ret i32 3
+
+sw.epilog:
+ ret i32 -1
+}
+
attributes #0 = { "frame-pointer"="all" }
More information about the llvm-commits
mailing list