mirror of
https://github.com/c64scene-ar/llvm-6502.git
synced 2024-11-05 13:09:10 +00:00
88e419d66e
Remove 'inlinedAt:' from MDLocalVariable. Besides saving some memory (variables with it seem to be single largest `Metadata` contributer to memory usage right now in -g -flto builds), this stops optimization and backend passes from having to change local variables. The 'inlinedAt:' field was used by the backend in two ways: 1. To tell the backend whether and into what a variable was inlined. 2. To create a unique id for each inlined variable. Instead, rely on the 'inlinedAt:' field of the intrinsic's `!dbg` attachment, and change the DWARF backend to use a typedef called `InlinedVariable` which is `std::pair<MDLocalVariable*, MDLocation*>`. This `DebugLoc` is already passed reliably through the backend (as verified by r234021). This commit removes the check from r234021, but I added a new check (that will survive) in r235048, and changed the `DIBuilder` API in r235041 to require a `!dbg` attachment whose 'scope:` is in the same `MDSubprogram` as the variable's. If this breaks your out-of-tree testcases, perhaps the script I used (mdlocalvariable-drop-inlinedat.sh) will help; I'll attach it to PR22778 in a moment. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@235050 91177308-0d34-0410-b5e6-96231b3b80d8
276 lines
13 KiB
LLVM
276 lines
13 KiB
LLVM
; REQUIRES: object-emission
|
|
|
|
; RUN: %llc_dwarf -filetype=obj -O0 < %s | llvm-dwarfdump -debug-dump=info - | FileCheck %s
|
|
|
|
; This isn't a very pretty test case - I imagine there might be other ways to
|
|
; tickle the optimizers into producing the desired code, but I haven't found
|
|
; them.
|
|
|
|
; The issue is when a function is inlined into itself, the inlined argument
|
|
; accidentally overwrote the concrete argument and was lost.
|
|
|
|
; IR generated from the following source compiled with clang -g:
|
|
; void fn1(void *);
|
|
; void fn2(int, int, int, int);
|
|
; void fn3();
|
|
; void fn8();
|
|
; struct C {
|
|
; int b;
|
|
; void m_fn2() {
|
|
; fn8();
|
|
; if (b) fn2(0, 0, 0, 0);
|
|
; fn3();
|
|
; }
|
|
; };
|
|
; C *x;
|
|
; inline void fn7() {}
|
|
; void fn6() {
|
|
; fn8();
|
|
; x->m_fn2();
|
|
; fn7();
|
|
; }
|
|
; void fn3() { fn6(); }
|
|
; void fn4() { x->m_fn2(); }
|
|
; void fn5() { x->m_fn2(); }
|
|
|
|
; The definition of C and declaration of C::m_fn2
|
|
; CHECK: DW_TAG_structure_type
|
|
; CHECK-NOT: {{DW_TAG|NULL}}
|
|
; CHECK: DW_TAG_member
|
|
; CHECK-NOT: {{DW_TAG|NULL}}
|
|
; CHECK: DW_TAG_subprogram
|
|
; CHECK-NOT: DW_TAG
|
|
; CHECK: DW_AT_name {{.*}} "m_fn2"
|
|
; CHECK-NOT: {{DW_TAG|NULL}}
|
|
; CHECK: [[M_FN2_THIS_DECL:.*]]: DW_TAG_formal_parameter
|
|
|
|
; The abstract definition of C::m_fn2
|
|
; CHECK: [[M_FN2_ABS_DEF:.*]]: DW_TAG_subprogram
|
|
; CHECK-NOT: DW_TAG
|
|
; CHECK: DW_AT_specification {{.*}} "_ZN1C5m_fn2Ev"
|
|
; CHECK-NOT: DW_TAG
|
|
; CHECK: DW_AT_inline
|
|
; CHECK-NOT: {{DW_TAG|NULL}}
|
|
; CHECK: [[M_FN2_THIS_ABS_DEF:.*]]: DW_TAG_formal_parameter
|
|
; CHECK-NOT: DW_TAG
|
|
; CHECK: DW_AT_name {{.*}} "this"
|
|
|
|
; Skip some other functions
|
|
; CHECK: DW_TAG_subprogram
|
|
; CHECK: DW_TAG_subprogram
|
|
; CHECK: DW_TAG_subprogram
|
|
|
|
; The concrete definition of C::m_fn2
|
|
; CHECK: DW_TAG_subprogram
|
|
; CHECK-NOT: DW_TAG
|
|
; CHECK: DW_AT_abstract_origin {{.*}} {[[M_FN2_ABS_DEF]]} "_ZN1C5m_fn2Ev"
|
|
; CHECK-NOT: {{DW_TAG|NULL}}
|
|
; CHECK: DW_TAG_formal_parameter
|
|
; CHECK-NOT: DW_TAG
|
|
; CHECK: DW_AT_abstract_origin {{.*}} {[[M_FN2_THIS_ABS_DEF]]}
|
|
; CHECK-NOT: {{DW_TAG|NULL}}
|
|
; Inlined fn3:
|
|
; CHECK: DW_TAG_inlined_subroutine
|
|
; CHECK-NOT: {{DW_TAG|NULL}}
|
|
; Inlined fn6:
|
|
; CHECK: DW_TAG_inlined_subroutine
|
|
; CHECK-NOT: {{DW_TAG|NULL}}
|
|
; Inlined C::m_fn2:
|
|
; CHECK: DW_TAG_inlined_subroutine
|
|
; CHECK-NOT: DW_TAG
|
|
; CHECK: DW_AT_abstract_origin {{.*}} {[[M_FN2_ABS_DEF]]} "_ZN1C5m_fn2Ev"
|
|
; CHECK-NOT: {{DW_TAG|NULL}}
|
|
; CHECK: DW_TAG_formal_parameter
|
|
; CHECK-NOT: DW_TAG
|
|
; CHECK: DW_AT_abstract_origin {{.*}} {[[M_FN2_THIS_ABS_DEF]]}
|
|
|
|
|
|
|
|
%struct.C = type { i32 }
|
|
|
|
@x = global %struct.C* null, align 8
|
|
|
|
; Function Attrs: nounwind
|
|
define void @_Z3fn6v() #0 {
|
|
entry:
|
|
tail call void @_Z3fn8v() #3, !dbg !31
|
|
%0 = load %struct.C*, %struct.C** @x, align 8, !dbg !32, !tbaa !33
|
|
tail call void @llvm.dbg.value(metadata %struct.C* %0, i64 0, metadata !37, metadata !MDExpression()) #3, !dbg !38
|
|
tail call void @_Z3fn8v() #3, !dbg !39
|
|
%b.i = getelementptr inbounds %struct.C, %struct.C* %0, i64 0, i32 0, !dbg !40
|
|
%1 = load i32, i32* %b.i, align 4, !dbg !40, !tbaa !42
|
|
%tobool.i = icmp eq i32 %1, 0, !dbg !40
|
|
br i1 %tobool.i, label %_ZN1C5m_fn2Ev.exit, label %if.then.i, !dbg !40
|
|
|
|
if.then.i: ; preds = %entry
|
|
tail call void @_Z3fn2iiii(i32 0, i32 0, i32 0, i32 0) #3, !dbg !45
|
|
br label %_ZN1C5m_fn2Ev.exit, !dbg !45
|
|
|
|
_ZN1C5m_fn2Ev.exit: ; preds = %entry, %if.then.i
|
|
tail call void @_Z3fn3v() #3, !dbg !47
|
|
ret void, !dbg !48
|
|
}
|
|
|
|
declare void @_Z3fn8v() #1
|
|
|
|
; Function Attrs: nounwind
|
|
define linkonce_odr void @_ZN1C5m_fn2Ev(%struct.C* nocapture readonly %this) #0 align 2 {
|
|
entry:
|
|
tail call void @llvm.dbg.value(metadata %struct.C* %this, i64 0, metadata !24, metadata !MDExpression()), !dbg !49
|
|
tail call void @_Z3fn8v() #3, !dbg !50
|
|
%b = getelementptr inbounds %struct.C, %struct.C* %this, i64 0, i32 0, !dbg !51
|
|
%0 = load i32, i32* %b, align 4, !dbg !51, !tbaa !42
|
|
%tobool = icmp eq i32 %0, 0, !dbg !51
|
|
br i1 %tobool, label %if.end, label %if.then, !dbg !51
|
|
|
|
if.then: ; preds = %entry
|
|
tail call void @_Z3fn2iiii(i32 0, i32 0, i32 0, i32 0) #3, !dbg !52
|
|
br label %if.end, !dbg !52
|
|
|
|
if.end: ; preds = %entry, %if.then
|
|
tail call void @_Z3fn8v() #3, !dbg !53
|
|
%1 = load %struct.C*, %struct.C** @x, align 8, !dbg !56, !tbaa !33
|
|
tail call void @llvm.dbg.value(metadata %struct.C* %1, i64 0, metadata !57, metadata !MDExpression()) #3, !dbg !58
|
|
tail call void @_Z3fn8v() #3, !dbg !59
|
|
%b.i.i = getelementptr inbounds %struct.C, %struct.C* %1, i64 0, i32 0, !dbg !60
|
|
%2 = load i32, i32* %b.i.i, align 4, !dbg !60, !tbaa !42
|
|
%tobool.i.i = icmp eq i32 %2, 0, !dbg !60
|
|
br i1 %tobool.i.i, label %_Z3fn6v.exit, label %if.then.i.i, !dbg !60
|
|
|
|
if.then.i.i: ; preds = %if.end
|
|
tail call void @_Z3fn2iiii(i32 0, i32 0, i32 0, i32 0) #3, !dbg !61
|
|
br label %_Z3fn6v.exit, !dbg !61
|
|
|
|
_Z3fn6v.exit: ; preds = %if.end, %if.then.i.i
|
|
tail call void @_Z3fn3v() #3, !dbg !62
|
|
ret void, !dbg !63
|
|
}
|
|
|
|
; Function Attrs: nounwind
|
|
define void @_Z3fn3v() #0 {
|
|
entry:
|
|
br label %tailrecurse
|
|
|
|
tailrecurse: ; preds = %tailrecurse.backedge, %entry
|
|
tail call void @_Z3fn8v() #3, !dbg !64
|
|
%0 = load %struct.C*, %struct.C** @x, align 8, !dbg !66, !tbaa !33
|
|
tail call void @llvm.dbg.value(metadata %struct.C* %0, i64 0, metadata !67, metadata !MDExpression()) #3, !dbg !68
|
|
tail call void @_Z3fn8v() #3, !dbg !69
|
|
%b.i.i = getelementptr inbounds %struct.C, %struct.C* %0, i64 0, i32 0, !dbg !70
|
|
%1 = load i32, i32* %b.i.i, align 4, !dbg !70, !tbaa !42
|
|
%tobool.i.i = icmp eq i32 %1, 0, !dbg !70
|
|
br i1 %tobool.i.i, label %tailrecurse.backedge, label %if.then.i.i, !dbg !70
|
|
|
|
tailrecurse.backedge: ; preds = %tailrecurse, %if.then.i.i
|
|
br label %tailrecurse
|
|
|
|
if.then.i.i: ; preds = %tailrecurse
|
|
tail call void @_Z3fn2iiii(i32 0, i32 0, i32 0, i32 0) #3, !dbg !71
|
|
br label %tailrecurse.backedge, !dbg !71
|
|
}
|
|
|
|
; Function Attrs: nounwind
|
|
define void @_Z3fn4v() #0 {
|
|
entry:
|
|
%0 = load %struct.C*, %struct.C** @x, align 8, !dbg !72, !tbaa !33
|
|
tail call void @_ZN1C5m_fn2Ev(%struct.C* %0), !dbg !72
|
|
ret void, !dbg !72
|
|
}
|
|
|
|
; Function Attrs: nounwind
|
|
define void @_Z3fn5v() #0 {
|
|
entry:
|
|
%0 = load %struct.C*, %struct.C** @x, align 8, !dbg !73, !tbaa !33
|
|
tail call void @_ZN1C5m_fn2Ev(%struct.C* %0), !dbg !73
|
|
ret void, !dbg !73
|
|
}
|
|
|
|
declare void @_Z3fn2iiii(i32, i32, i32, i32) #1
|
|
|
|
; Function Attrs: nounwind readnone
|
|
declare void @llvm.dbg.value(metadata, i64, metadata, metadata) #2
|
|
|
|
attributes #0 = { nounwind "less-precise-fpmad"="false" "no-frame-pointer-elim"="false" "no-infs-fp-math"="false" "no-nans-fp-math"="false" "no-realign-stack" "stack-protector-buffer-size"="8" "unsafe-fp-math"="false" "use-soft-float"="false" }
|
|
attributes #1 = { "less-precise-fpmad"="false" "no-frame-pointer-elim"="false" "no-infs-fp-math"="false" "no-nans-fp-math"="false" "no-realign-stack" "stack-protector-buffer-size"="8" "unsafe-fp-math"="false" "use-soft-float"="false" }
|
|
attributes #2 = { nounwind readnone }
|
|
attributes #3 = { nounwind }
|
|
|
|
!llvm.dbg.cu = !{!0}
|
|
!llvm.module.flags = !{!28, !29}
|
|
!llvm.ident = !{!30}
|
|
|
|
!0 = !MDCompileUnit(language: DW_LANG_C_plus_plus, producer: "clang version 3.6.0 ", isOptimized: true, emissionKind: 1, file: !1, enums: !2, retainedTypes: !3, subprograms: !13, globals: !26, imports: !2)
|
|
!1 = !MDFile(filename: "<stdin>", directory: "/usr/local/google/home/blaikie/dev/scratch/missing_concrete_variable_on_darwin/reduce")
|
|
!2 = !{}
|
|
!3 = !{!4}
|
|
!4 = !MDCompositeType(tag: DW_TAG_structure_type, name: "C", line: 5, size: 32, align: 32, file: !5, elements: !6, identifier: "_ZTS1C")
|
|
!5 = !MDFile(filename: "recursive_inlining.cpp", directory: "/usr/local/google/home/blaikie/dev/scratch/missing_concrete_variable_on_darwin/reduce")
|
|
!6 = !{!7, !9}
|
|
!7 = !MDDerivedType(tag: DW_TAG_member, name: "b", line: 6, size: 32, align: 32, file: !5, scope: !"_ZTS1C", baseType: !8)
|
|
!8 = !MDBasicType(tag: DW_TAG_base_type, name: "int", size: 32, align: 32, encoding: DW_ATE_signed)
|
|
!9 = !MDSubprogram(name: "m_fn2", linkageName: "_ZN1C5m_fn2Ev", line: 7, isLocal: false, isDefinition: false, virtualIndex: 6, flags: DIFlagPrototyped, isOptimized: true, scopeLine: 7, file: !5, scope: !"_ZTS1C", type: !10)
|
|
!10 = !MDSubroutineType(types: !11)
|
|
!11 = !{null, !12}
|
|
!12 = !MDDerivedType(tag: DW_TAG_pointer_type, size: 64, align: 64, flags: DIFlagArtificial | DIFlagObjectPointer, baseType: !"_ZTS1C")
|
|
!13 = !{!14, !18, !19, !20, !21, !22}
|
|
!14 = !MDSubprogram(name: "fn6", linkageName: "_Z3fn6v", line: 15, isLocal: false, isDefinition: true, virtualIndex: 6, flags: DIFlagPrototyped, isOptimized: true, scopeLine: 15, file: !5, scope: !15, type: !16, function: void ()* @_Z3fn6v, variables: !2)
|
|
!15 = !MDFile(filename: "recursive_inlining.cpp", directory: "/usr/local/google/home/blaikie/dev/scratch/missing_concrete_variable_on_darwin/reduce")
|
|
!16 = !MDSubroutineType(types: !17)
|
|
!17 = !{null}
|
|
!18 = !MDSubprogram(name: "fn3", linkageName: "_Z3fn3v", line: 20, isLocal: false, isDefinition: true, virtualIndex: 6, flags: DIFlagPrototyped, isOptimized: true, scopeLine: 20, file: !5, scope: !15, type: !16, function: void ()* @_Z3fn3v, variables: !2)
|
|
!19 = !MDSubprogram(name: "fn4", linkageName: "_Z3fn4v", line: 21, isLocal: false, isDefinition: true, virtualIndex: 6, flags: DIFlagPrototyped, isOptimized: true, scopeLine: 21, file: !5, scope: !15, type: !16, function: void ()* @_Z3fn4v, variables: !2)
|
|
!20 = !MDSubprogram(name: "fn5", linkageName: "_Z3fn5v", line: 22, isLocal: false, isDefinition: true, virtualIndex: 6, flags: DIFlagPrototyped, isOptimized: true, scopeLine: 22, file: !5, scope: !15, type: !16, function: void ()* @_Z3fn5v, variables: !2)
|
|
!21 = !MDSubprogram(name: "fn7", linkageName: "_Z3fn7v", line: 14, isLocal: false, isDefinition: true, virtualIndex: 6, flags: DIFlagPrototyped, isOptimized: true, scopeLine: 14, file: !5, scope: !15, type: !16, variables: !2)
|
|
!22 = !MDSubprogram(name: "m_fn2", linkageName: "_ZN1C5m_fn2Ev", line: 7, isLocal: false, isDefinition: true, virtualIndex: 6, flags: DIFlagPrototyped, isOptimized: true, scopeLine: 7, file: !5, scope: !"_ZTS1C", type: !10, function: void (%struct.C*)* @_ZN1C5m_fn2Ev, declaration: !9, variables: !23)
|
|
!23 = !{!24}
|
|
!24 = !MDLocalVariable(tag: DW_TAG_arg_variable, name: "this", arg: 1, flags: DIFlagArtificial | DIFlagObjectPointer, scope: !22, type: !25)
|
|
!25 = !MDDerivedType(tag: DW_TAG_pointer_type, size: 64, align: 64, baseType: !"_ZTS1C")
|
|
!26 = !{!27}
|
|
!27 = !MDGlobalVariable(name: "x", line: 13, isLocal: false, isDefinition: true, scope: null, file: !15, type: !25, variable: %struct.C** @x)
|
|
!28 = !{i32 2, !"Dwarf Version", i32 4}
|
|
!29 = !{i32 2, !"Debug Info Version", i32 3}
|
|
!30 = !{!"clang version 3.6.0 "}
|
|
!31 = !MDLocation(line: 16, scope: !14)
|
|
!32 = !MDLocation(line: 17, scope: !14)
|
|
!33 = !{!34, !34, i64 0}
|
|
!34 = !{!"any pointer", !35, i64 0}
|
|
!35 = !{!"omnipotent char", !36, i64 0}
|
|
!36 = !{!"Simple C/C++ TBAA"}
|
|
!37 = !MDLocalVariable(tag: DW_TAG_arg_variable, name: "this", arg: 1, flags: DIFlagArtificial | DIFlagObjectPointer, scope: !22, type: !25)
|
|
!38 = !MDLocation(line: 0, scope: !22, inlinedAt: !32)
|
|
!39 = !MDLocation(line: 8, scope: !22, inlinedAt: !32)
|
|
!40 = !MDLocation(line: 9, scope: !41, inlinedAt: !32)
|
|
!41 = distinct !MDLexicalBlock(line: 9, column: 0, file: !5, scope: !22)
|
|
!42 = !{!43, !44, i64 0}
|
|
!43 = !{!"_ZTS1C", !44, i64 0}
|
|
!44 = !{!"int", !35, i64 0}
|
|
!45 = !MDLocation(line: 9, scope: !46, inlinedAt: !32)
|
|
!46 = distinct !MDLexicalBlock(line: 9, column: 0, file: !5, scope: !41)
|
|
!47 = !MDLocation(line: 10, scope: !22, inlinedAt: !32)
|
|
!48 = !MDLocation(line: 19, scope: !14)
|
|
!49 = !MDLocation(line: 0, scope: !22)
|
|
!50 = !MDLocation(line: 8, scope: !22)
|
|
!51 = !MDLocation(line: 9, scope: !41)
|
|
!52 = !MDLocation(line: 9, scope: !46)
|
|
!53 = !MDLocation(line: 16, scope: !14, inlinedAt: !54)
|
|
!54 = !MDLocation(line: 20, scope: !18, inlinedAt: !55)
|
|
!55 = !MDLocation(line: 10, scope: !22)
|
|
!56 = !MDLocation(line: 17, scope: !14, inlinedAt: !54)
|
|
!57 = !MDLocalVariable(tag: DW_TAG_arg_variable, name: "this", arg: 1, flags: DIFlagArtificial | DIFlagObjectPointer, scope: !22, type: !25)
|
|
!58 = !MDLocation(line: 0, scope: !22, inlinedAt: !56)
|
|
!59 = !MDLocation(line: 8, scope: !22, inlinedAt: !56)
|
|
!60 = !MDLocation(line: 9, scope: !41, inlinedAt: !56)
|
|
!61 = !MDLocation(line: 9, scope: !46, inlinedAt: !56)
|
|
!62 = !MDLocation(line: 10, scope: !22, inlinedAt: !56)
|
|
!63 = !MDLocation(line: 11, scope: !22)
|
|
!64 = !MDLocation(line: 16, scope: !14, inlinedAt: !65)
|
|
!65 = !MDLocation(line: 20, scope: !18)
|
|
!66 = !MDLocation(line: 17, scope: !14, inlinedAt: !65)
|
|
!67 = !MDLocalVariable(tag: DW_TAG_arg_variable, name: "this", arg: 1, flags: DIFlagArtificial | DIFlagObjectPointer, scope: !22, type: !25)
|
|
!68 = !MDLocation(line: 0, scope: !22, inlinedAt: !66)
|
|
!69 = !MDLocation(line: 8, scope: !22, inlinedAt: !66)
|
|
!70 = !MDLocation(line: 9, scope: !41, inlinedAt: !66)
|
|
!71 = !MDLocation(line: 9, scope: !46, inlinedAt: !66)
|
|
!72 = !MDLocation(line: 21, scope: !19)
|
|
!73 = !MDLocation(line: 22, scope: !20)
|