[llvm] r288450 - [DWARF] Put linkage-name on abstract origin even when there's a declaration.
Paul Robinson via llvm-commits
llvm-commits at lists.llvm.org
Thu Dec 1 17:55:17 PST 2016
Author: probinson
Date: Thu Dec 1 19:55:17 2016
New Revision: 288450
URL: http://llvm.org/viewvc/llvm-project?rev=288450&view=rev
Log:
[DWARF] Put linkage-name on abstract origin even when there's a declaration.
In r266692, we made it possible to emit linkage names for just inlined
functions, putting the attribute on the abstract origin. Make sure we
don't think the linkage-name was already emitted on a declaration.
Differential Revision: http://reviews.llvm.org/D27320
Modified:
llvm/trunk/lib/CodeGen/AsmPrinter/DwarfUnit.cpp
llvm/trunk/test/DebugInfo/Generic/linkage-name-abstract.ll
Modified: llvm/trunk/lib/CodeGen/AsmPrinter/DwarfUnit.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/AsmPrinter/DwarfUnit.cpp?rev=288450&r1=288449&r2=288450&view=diff
==============================================================================
--- llvm/trunk/lib/CodeGen/AsmPrinter/DwarfUnit.cpp (original)
+++ llvm/trunk/lib/CodeGen/AsmPrinter/DwarfUnit.cpp Thu Dec 1 19:55:17 2016
@@ -1150,7 +1150,9 @@ bool DwarfUnit::applySubprogramDefinitio
assert(DeclDie && "This DIE should've already been constructed when the "
"definition DIE was created in "
"getOrCreateSubprogramDIE");
- DeclLinkageName = SPDecl->getLinkageName();
+ // Look at the Decl's linkage name only if we emitted it.
+ if (DD->useAllLinkageNames())
+ DeclLinkageName = SPDecl->getLinkageName();
unsigned DeclID =
getOrCreateSourceID(SPDecl->getFilename(), SPDecl->getDirectory());
unsigned DefID = getOrCreateSourceID(SP->getFilename(), SP->getDirectory());
Modified: llvm/trunk/test/DebugInfo/Generic/linkage-name-abstract.ll
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/DebugInfo/Generic/linkage-name-abstract.ll?rev=288450&r1=288449&r2=288450&view=diff
==============================================================================
--- llvm/trunk/test/DebugInfo/Generic/linkage-name-abstract.ll (original)
+++ llvm/trunk/test/DebugInfo/Generic/linkage-name-abstract.ll Thu Dec 1 19:55:17 2016
@@ -18,56 +18,115 @@
; void f3() {
; f2();
; }
+;
+; struct F4 {
+; __attribute__((always_inline)) void f5();
+; };
+; void F4::f5() {
+; f1();
+; }
+; void f6() {
+; F4::f5();
+; }
-; Show that there's only one linkage_name.
-; ONENAME: {{DW_AT(_MIPS)?_linkage_name}}
+; Show that the only linkage names are for the inlined functions,
+; because those are the ones with an abstract origin.
+; ONENAME-NOT: {{DW_AT(_MIPS)?_linkage_name}}
+; ONENAME: {{DW_AT(_MIPS)?_linkage_name}} {{.*}} "_Z2f2v"
+; ONENAME-NOT: {{DW_AT(_MIPS)?_linkage_name}}
+; ONENAME: {{DW_AT(_MIPS)?_linkage_name}} {{.*}} "_ZN2F42f5Ev"
; ONENAME-NOT: {{DW_AT(_MIPS)?_linkage_name}}
-; Locate the subprogram DIE with the linkage name.
-; Show that the inlined_subroutine refers to it.
-; REF: DW_TAG_subprogram
-; REF: [[FOO:0x.*]]: DW_TAG_subprogram
-; REF-NOT: {{DW_TAG|NULL}}
-; REF: {{DW_AT(_MIPS)?_linkage_name}}
-; REF: DW_TAG_inlined_subroutine
-; REF-NOT: {{DW_TAG|NULL}}
-; REF: DW_AT_abstract_origin {{.*}} {[[FOO]]}
+; For f2() we see the definition pointing to an abstract origin DIE,
+; which in turn is where the linkage_name is; and then there's
+; an inlined_subroutine pointing back to the abstract origin.
+; The order of these DIEs is not important of course, just the links.
+; REF: DW_TAG_subprogram
+; REF-NOT: {{DW_TAG|NULL}}
+; REF: DW_AT_abstract_origin {{.*}} {[[F2:0x.*]]} "_Z2f2v"
+; REF: [[F2]]: DW_TAG_subprogram
+; REF-NEXT: linkage_name {{.*}} "_Z2f2v"
+; REF: DW_TAG_inlined_subroutine
+; REF-NOT: {{DW_TAG|NULL}}
+; REF: DW_AT_abstract_origin {{.*}} {[[F2]]}
+
+; For F4::f5(), first we see the in-class declaration,
+; then the definition, abstract origin, and the inlined_subroutine.
+; REF: DW_TAG_structure_type
+; REF-NEXT: DW_AT_name {{.*}} "F4"
+; REF-NOT: {{DW_TAG|NULL}}
+; REF: [[F5_DECL:0x.*]]: DW_TAG_subprogram
+; REF-NEXT: DW_AT_name {{.*}} "f5"
+; REF: DW_TAG_subprogram
+; REF-NOT: {{DW_TAG|NULL}}
+; REF: DW_AT_abstract_origin {{.*}} {[[F5_ABS:0x.*]]} "_ZN2F42f5Ev"
+; REF: [[F5_ABS]]: DW_TAG_subprogram
+; REF-NOT: {{DW_TAG|NULL}}
+; REF: linkage_name {{.*}} "_ZN2F42f5Ev"
+; REF-NEXT: DW_AT_specification {{.*}} {[[F5_DECL]]}
+; REF: DW_TAG_inlined_subroutine
+; REF-NOT: {{DW_TAG|NULL}}
+; REF: DW_AT_abstract_origin {{.*}} {[[F5_ABS]]}
+
; Function Attrs: alwaysinline uwtable
-define void @_Z2f2v() #0 !dbg !4 {
+define void @_Z2f2v() #0 !dbg !6 {
entry:
- call void @_Z2f1v(), !dbg !11
- ret void, !dbg !12
+ call void @_Z2f1v(), !dbg !9
+ ret void, !dbg !10
}
declare void @_Z2f1v()
; Function Attrs: uwtable
-define void @_Z2f3v() #2 !dbg !7 {
+define void @_Z2f3v() !dbg !11 {
+entry:
+ call void @_Z2f1v(), !dbg !12
+ ret void, !dbg !14
+}
+
+; Function Attrs: alwaysinline uwtable
+define void @_ZN2F42f5Ev() #0 align 2 !dbg !15 {
+entry:
+ call void @_Z2f1v(), !dbg !19
+ ret void, !dbg !20
+}
+
+; Function Attrs: uwtable
+define void @_Z2f6v() !dbg !21 {
entry:
- call void @_Z2f1v(), !dbg !13
- ret void, !dbg !15
+ call void @_Z2f1v(), !dbg !22
+ ret void, !dbg !24
}
-attributes #0 = { alwaysinline uwtable }
-attributes #2 = { uwtable }
+attributes #0 = { alwaysinline }
!llvm.dbg.cu = !{!0}
-!llvm.module.flags = !{!8, !9}
-!llvm.ident = !{!10}
+!llvm.module.flags = !{!3, !4}
+!llvm.ident = !{!5}
-!0 = distinct !DICompileUnit(language: DW_LANG_C_plus_plus, file: !1, producer: "clang version 3.9.0 (trunk 265282)", isOptimized: false, runtimeVersion: 0, emissionKind: FullDebug, enums: !2)
-!1 = !DIFile(filename: "linkage-name-abstract.cpp", directory: "/home/probinson/projects/scratch")
+!0 = distinct !DICompileUnit(language: DW_LANG_C_plus_plus, file: !1, producer: "clang version 4.0.0 (trunk 288231)", isOptimized: false, runtimeVersion: 0, emissionKind: FullDebug, enums: !2)
+!1 = !DIFile(filename: "linkage-name-abstract-static.cpp", directory: "/home/probinson/projects/scratch")
!2 = !{}
-!4 = distinct !DISubprogram(name: "f2", linkageName: "_Z2f2v", scope: !1, file: !1, line: 2, type: !5, isLocal: false, isDefinition: true, scopeLine: 2, flags: DIFlagPrototyped, isOptimized: false, unit: !0, variables: !2)
-!5 = !DISubroutineType(types: !6)
-!6 = !{null}
-!7 = distinct !DISubprogram(name: "f3", linkageName: "_Z2f3v", scope: !1, file: !1, line: 5, type: !5, isLocal: false, isDefinition: true, scopeLine: 5, flags: DIFlagPrototyped, isOptimized: false, unit: !0, variables: !2)
-!8 = !{i32 2, !"Dwarf Version", i32 4}
-!9 = !{i32 2, !"Debug Info Version", i32 3}
-!10 = !{!"clang version 3.9.0 (trunk 265282)"}
-!11 = !DILocation(line: 3, column: 3, scope: !4)
-!12 = !DILocation(line: 4, column: 1, scope: !4)
-!13 = !DILocation(line: 3, column: 3, scope: !4, inlinedAt: !14)
-!14 = distinct !DILocation(line: 6, column: 3, scope: !7)
-!15 = !DILocation(line: 7, column: 1, scope: !7)
+!3 = !{i32 2, !"Dwarf Version", i32 4}
+!4 = !{i32 2, !"Debug Info Version", i32 3}
+!5 = !{!"clang version 4.0.0 (trunk 288231)"}
+!6 = distinct !DISubprogram(name: "f2", linkageName: "_Z2f2v", scope: !1, file: !1, line: 2, type: !7, isLocal: false, isDefinition: true, scopeLine: 2, flags: DIFlagPrototyped, isOptimized: false, unit: !0, variables: !2)
+!7 = !DISubroutineType(types: !8)
+!8 = !{null}
+!9 = !DILocation(line: 3, column: 3, scope: !6)
+!10 = !DILocation(line: 4, column: 1, scope: !6)
+!11 = distinct !DISubprogram(name: "f3", linkageName: "_Z2f3v", scope: !1, file: !1, line: 5, type: !7, isLocal: false, isDefinition: true, scopeLine: 5, flags: DIFlagPrototyped, isOptimized: false, unit: !0, variables: !2)
+!12 = !DILocation(line: 3, column: 3, scope: !6, inlinedAt: !13)
+!13 = distinct !DILocation(line: 6, column: 3, scope: !11)
+!14 = !DILocation(line: 7, column: 1, scope: !11)
+!15 = distinct !DISubprogram(name: "f5", linkageName: "_ZN2F42f5Ev", scope: !16, file: !1, line: 12, type: !7, isLocal: false, isDefinition: true, scopeLine: 12, flags: DIFlagPrototyped, isOptimized: false, unit: !0, declaration: !18, variables: !2)
+!16 = distinct !DICompositeType(tag: DW_TAG_structure_type, name: "F4", file: !1, line: 9, size: 8, elements: !17, identifier: "_ZTS2F4")
+!17 = !{!18}
+!18 = !DISubprogram(name: "f5", linkageName: "_ZN2F42f5Ev", scope: !16, file: !1, line: 10, type: !7, isLocal: false, isDefinition: false, scopeLine: 10, flags: DIFlagPrototyped, isOptimized: false)
+!19 = !DILocation(line: 13, column: 3, scope: !15)
+!20 = !DILocation(line: 14, column: 1, scope: !15)
+!21 = distinct !DISubprogram(name: "f6", linkageName: "_Z2f6v", scope: !1, file: !1, line: 15, type: !7, isLocal: false, isDefinition: true, scopeLine: 15, flags: DIFlagPrototyped, isOptimized: false, unit: !0, variables: !2)
+!22 = !DILocation(line: 13, column: 3, scope: !15, inlinedAt: !23)
+!23 = distinct !DILocation(line: 16, column: 3, scope: !21)
+!24 = !DILocation(line: 17, column: 1, scope: !21)
More information about the llvm-commits
mailing list