[llvm] b7c6697 - [DebugInfo] Add tests that we emit .eh_frame instead of .debug_frame

[Fangrui Song via llvm-commits]

Author: Fangrui Song
Date: 2021-04-29T09:35:48-07:00
New Revision: b7c6697813af21fc2bf1a87e1d66d79caea3d790

URL: https://github.com/llvm/llvm-project/commit/b7c6697813af21fc2bf1a87e1d66d79caea3d790
DIFF: https://github.com/llvm/llvm-project/commit/b7c6697813af21fc2bf1a87e1d66d79caea3d790.diff

LOG: [DebugInfo] Add tests that we emit .eh_frame instead of .debug_frame

Add tests which can catch the issue in 0ce723cb228bc1d1a0f5718f3862fb836145a333
(If any function needs CFISection::EH, the module should use CFISection::EH).

Reviewed By: echristo

Differential Revision: https://reviews.llvm.org/D101339

Added: 
    llvm/test/DebugInfo/X86/cfi_sections.ll

Modified: 
    

Removed: 
    llvm/test/DebugInfo/X86/debug_frame.ll


################################################################################
diff  --git a/llvm/test/DebugInfo/X86/cfi_sections.ll b/llvm/test/DebugInfo/X86/cfi_sections.ll
new file mode 100644
index 0000000000000..c7bd9620d51c2
--- /dev/null
+++ b/llvm/test/DebugInfo/X86/cfi_sections.ll
@@ -0,0 +1,96 @@
+; RUN: split-file %s %t
+; RUN: llc < %t/1.ll -mtriple=x86_64 | FileCheck %s --check-prefix=DEBUG_FRAME
+; RUN: llc < %t/1.ll -mtriple=x86_64 -force-dwarf-frame-section | FileCheck %s --check-prefix=DEBUG_FRAME
+; RUN: llc < %t/2.ll -mtriple=x86_64 | FileCheck %s --check-prefix=EH_FRAME
+; RUN: llc < %t/2.ll -mtriple=x86_64 -force-dwarf-frame-section | FileCheck %s --check-prefix=BOTH
+; RUN: llc < %t/3.ll -mtriple=x86_64 | FileCheck %s --check-prefix=EH_FRAME
+; RUN: llc < %t/3.ll -mtriple=x86_64 -force-dwarf-frame-section | FileCheck %s --check-prefix=BOTH
+
+; EH_FRAME-NOT: .cfi_sections
+; DEBUG_FRAME: .cfi_sections .debug_frame{{$}}
+; BOTH: .cfi_sections .eh_frame, .debug_frame{{$}}
+
+;--- 1.ll
+;; No function has uwtable/personality or throws && f needs debug info -> emit .debug_frame
+
+define void @f() nounwind !dbg !6 {
+entry:
+  ret void
+}
+
+define void @g() nounwind {
+entry:
+  ret void
+}
+
+!llvm.dbg.cu = !{!0}
+!llvm.module.flags = !{!3, !4, !5}
+
+!0 = distinct !DICompileUnit(language: DW_LANG_C99, file: !1, producer: "clang version 13.0.0", isOptimized: true, runtimeVersion: 0, emissionKind: FullDebug, enums: !2, splitDebugInlining: false, nameTableKind: None)
+!1 = !DIFile(filename: "a.c", directory: "/tmp")
+!2 = !{}
+!3 = !{i32 7, !"Dwarf Version", i32 5}
+!4 = !{i32 2, !"Debug Info Version", i32 3}
+!5 = !{i32 1, !"wchar_size", i32 4}
+!6 = distinct !DISubprogram(name: "f", scope: !1, file: !1, line: 1, type: !7, scopeLine: 1, flags: DIFlagPrototyped | DIFlagAllCallsDescribed, spFlags: DISPFlagDefinition | DISPFlagOptimized, unit: !0, retainedNodes: !2)
+!7 = !DISubroutineType(types: !8)
+!8 = !{null}
+!9 = !DILocation(line: 1, column: 15, scope: !6)
+
+;--- 2.ll
+;; g has uwtable and thus needs .eh_frame
+
+define void @f() nounwind !dbg !6 {
+entry:
+  ret void
+}
+
+define void @g() nounwind uwtable !dbg !10 {
+entry:
+  ret void
+}
+
+!llvm.dbg.cu = !{!0}
+!llvm.module.flags = !{!3, !4, !5}
+
+!0 = distinct !DICompileUnit(language: DW_LANG_C99, file: !1, producer: "clang version 13.0.0", isOptimized: true, runtimeVersion: 0, emissionKind: FullDebug, enums: !2, splitDebugInlining: false, nameTableKind: None)
+!1 = !DIFile(filename: "a.c", directory: "/tmp")
+!2 = !{}
+!3 = !{i32 7, !"Dwarf Version", i32 5}
+!4 = !{i32 2, !"Debug Info Version", i32 3}
+!5 = !{i32 1, !"wchar_size", i32 4}
+!6 = distinct !DISubprogram(name: "f", scope: !1, file: !1, line: 1, type: !7, scopeLine: 1, flags: DIFlagPrototyped | DIFlagAllCallsDescribed, spFlags: DISPFlagDefinition | DISPFlagOptimized, unit: !0, retainedNodes: !2)
+!7 = !DISubroutineType(types: !8)
+!8 = !{null}
+!9 = !DILocation(line: 1, column: 15, scope: !6)
+!10 = distinct !DISubprogram(name: "g", scope: !1, file: !1, line: 2, type: !7, scopeLine: 2, flags: DIFlagPrototyped | DIFlagAllCallsDescribed, spFlags: DISPFlagDefinition | DISPFlagOptimized, unit: !0, retainedNodes: !2)
+!11 = !DILocation(line: 2, column: 15, scope: !10)
+
+;--- 3.ll
+;; g has no nounwind, so it is treated as throwable. Emit .eh_frame.
+
+define void @f() nounwind !dbg !6 {
+entry:
+  ret void
+}
+
+define void @g() !dbg !10 {
+entry:
+  ret void
+}
+
+!llvm.dbg.cu = !{!0}
+!llvm.module.flags = !{!3, !4, !5}
+
+!0 = distinct !DICompileUnit(language: DW_LANG_C99, file: !1, producer: "clang version 13.0.0", isOptimized: true, runtimeVersion: 0, emissionKind: FullDebug, enums: !2, splitDebugInlining: false, nameTableKind: None)
+!1 = !DIFile(filename: "a.c", directory: "/tmp")
+!2 = !{}
+!3 = !{i32 7, !"Dwarf Version", i32 5}
+!4 = !{i32 2, !"Debug Info Version", i32 3}
+!5 = !{i32 1, !"wchar_size", i32 4}
+!6 = distinct !DISubprogram(name: "f", scope: !1, file: !1, line: 1, type: !7, scopeLine: 1, flags: DIFlagPrototyped | DIFlagAllCallsDescribed, spFlags: DISPFlagDefinition | DISPFlagOptimized, unit: !0, retainedNodes: !2)
+!7 = !DISubroutineType(types: !8)
+!8 = !{null}
+!9 = !DILocation(line: 1, column: 15, scope: !6)
+!10 = distinct !DISubprogram(name: "g", scope: !1, file: !1, line: 2, type: !7, scopeLine: 2, flags: DIFlagPrototyped | DIFlagAllCallsDescribed, spFlags: DISPFlagDefinition | DISPFlagOptimized, unit: !0, retainedNodes: !2)
+!11 = !DILocation(line: 2, column: 15, scope: !10)

diff  --git a/llvm/test/DebugInfo/X86/debug_frame.ll b/llvm/test/DebugInfo/X86/debug_frame.ll
deleted file mode 100644
index a1f0b08292b37..0000000000000
--- a/llvm/test/DebugInfo/X86/debug_frame.ll
+++ /dev/null
@@ -1,22 +0,0 @@
-; RUN: llc %s -mtriple=i686-pc-linux-gnu -o - | FileCheck %s
-
-; Test that we produce a .debug_frame, not an .eh_frame
-
-; CHECK: .cfi_sections .debug_frame
-
-define void @f() nounwind !dbg !0 {
-entry:
-  ret void
-}
-
-!llvm.dbg.cu = !{!2}
-!llvm.module.flags = !{!7}
-!5 = !{!0}
-
-!0 = distinct !DISubprogram(name: "f", line: 1, isLocal: false, isDefinition: true, virtualIndex: 6, flags: DIFlagPrototyped, isOptimized: true, unit: !2, scopeLine: 1, file: !6, scope: !1, type: !3)
-!1 = !DIFile(filename: "/home/espindola/llvm/test.c", directory: "/home/espindola/llvm/build")
-!2 = distinct !DICompileUnit(language: DW_LANG_C99, producer: "clang version 3.0 ()", isOptimized: true, emissionKind: FullDebug, file: !6, enums: !{}, retainedTypes: !{})
-!3 = !DISubroutineType(types: !4)
-!4 = !{null}
-!6 = !DIFile(filename: "/home/espindola/llvm/test.c", directory: "/home/espindola/llvm/build")
-!7 = !{i32 1, !"Debug Info Version", i32 3}