mirror of
https://github.com/c64scene-ar/llvm-6502.git
synced 2024-12-28 19:31:58 +00:00
f93662d3d5
When two calls from the same MDLocation are inlined they currently get treated as one inlined function call (creating difficulty debugging, duplicate variables, etc). Clang worked around this by including column information on inline calls which doesn't address LTO inlining or calls to the same function from the same line and column (such as through a macro). It also didn't address ctor and member function calls. By making the inlinedAt locations distinct, every call site has an explicitly distinct location that cannot be coalesced with any other call. This can produce linearly (2x in the worst case where every call is inlined and the call instruction has a non-call instruction at the same location) more debug locations. Any increase beyond that are in cases where the Clang workaround was insufficient and the new scheme is creating necessary distinct nodes that were being erroneously coalesced previously. After this change to LLVM the incomplete workarounds in Clang. That should reduce the number of debug locations (in a build without column info, the default on Darwin, not the default on Linux) by not creating pseudo-distinct locations for every call to an inline function. (oh, and I made the inlined-at chain rebuilding iterative instead of recursive because I was having trouble wrapping my head around it the way it was - open to discussion on the right design for that function (including going back to a recursive solution)) git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@226736 91177308-0d34-0410-b5e6-96231b3b80d8
122 lines
5.7 KiB
LLVM
122 lines
5.7 KiB
LLVM
; RUN: opt < %s -always-inline -S | FileCheck %s
|
|
|
|
; Original input generated from clang -emit-llvm -S -c -mllvm -disable-llvm-optzns
|
|
;
|
|
; #define CALLS1 f2(); f2();
|
|
; #define CALLS2 f4(); f4();
|
|
; void f1();
|
|
; inline __attribute__((always_inline)) void f2() {
|
|
; f1();
|
|
; }
|
|
; inline __attribute__((always_inline)) void f3() {
|
|
; CALLS1
|
|
; }
|
|
; inline __attribute__((always_inline)) void f4() {
|
|
; f3();
|
|
; }
|
|
; void f() {
|
|
; CALLS2
|
|
; }
|
|
|
|
; There should be unique locations for all 4 of these instructions, correctly
|
|
; describing the inlining that has occurred, even in the face of duplicate call
|
|
; site locations.
|
|
|
|
; The nomenclature used for the tags here is <function name>[cs<number>] where
|
|
; 'cs' is an abbreviation for 'call site' and the number indicates which call
|
|
; site from within the named function this is. (so, given the above inlining, we
|
|
; should have 4 calls to 'f1', two from the first call to f4 and two from the
|
|
; second call to f4)
|
|
|
|
; CHECK: call void @_Z2f1v(), !dbg [[fcs1_f4_f3cs1_f2:![0-9]+]]
|
|
; CHECK: call void @_Z2f1v(), !dbg [[fcs1_f4_f3cs2_f2:![0-9]+]]
|
|
; CHECK: call void @_Z2f1v(), !dbg [[fcs2_f4_f3cs1_f2:![0-9]+]]
|
|
; CHECK: call void @_Z2f1v(), !dbg [[fcs2_f4_f3cs2_f2:![0-9]+]]
|
|
|
|
; CHECK-DAG: [[F:![0-9]+]] = {{.*}} ; [ DW_TAG_subprogram ] {{.*}} [f]
|
|
; CHECK-DAG: [[F2:![0-9]+]] = {{.*}} ; [ DW_TAG_subprogram ] {{.*}} [f2]
|
|
; CHECK-DAG: [[F3:![0-9]+]] = {{.*}} ; [ DW_TAG_subprogram ] {{.*}} [f3]
|
|
; CHECK-DAG: [[F4:![0-9]+]] = {{.*}} ; [ DW_TAG_subprogram ] {{.*}} [f4]
|
|
|
|
; CHECK: [[fcs1_f4_f3cs1_f2]] = {{.*}}, scope: [[F2]], inlinedAt: [[fcs1_f4_f3cs1:![0-9]+]])
|
|
; CHECK: [[fcs1_f4_f3cs1]] = {{.*}}, scope: [[F3]], inlinedAt: [[fcs1_f4:![0-9]+]])
|
|
; CHECK: [[fcs1_f4]] = {{.*}}, scope: [[F4]], inlinedAt: [[fcs1:![0-9]+]])
|
|
; CHECK: [[fcs1]] = {{.*}}, scope: [[F]])
|
|
; CHECK: [[fcs1_f4_f3cs2_f2]] = {{.*}}, scope: [[F2]], inlinedAt: [[fcs1_f4_f3cs2:![0-9]+]])
|
|
; CHECK: [[fcs1_f4_f3cs2]] = {{.*}}, scope: [[F3]], inlinedAt: [[fcs1_f4]])
|
|
|
|
; CHECK: [[fcs2_f4_f3cs1_f2]] = {{.*}}, scope: [[F2]], inlinedAt: [[fcs2_f4_f3cs1:![0-9]+]])
|
|
; CHECK: [[fcs2_f4_f3cs1]] = {{.*}}, scope: [[F3]], inlinedAt: [[fcs2_f4:![0-9]+]])
|
|
; CHECK: [[fcs2_f4]] = {{.*}}, scope: [[F4]], inlinedAt: [[fcs2:![0-9]+]])
|
|
; CHECK: [[fcs2]] = {{.*}}, scope: [[F]])
|
|
; CHECK: [[fcs2_f4_f3cs2_f2]] = {{.*}}, scope: [[F2]], inlinedAt: [[fcs2_f4_f3cs2:![0-9]+]])
|
|
; CHECK: [[fcs2_f4_f3cs2]] = {{.*}}, scope: [[F3]], inlinedAt: [[fcs2_f4]])
|
|
|
|
$_Z2f4v = comdat any
|
|
|
|
$_Z2f3v = comdat any
|
|
|
|
$_Z2f2v = comdat any
|
|
|
|
; Function Attrs: uwtable
|
|
define void @_Z1fv() #0 {
|
|
entry:
|
|
call void @_Z2f4v(), !dbg !13
|
|
call void @_Z2f4v(), !dbg !13
|
|
ret void, !dbg !14
|
|
}
|
|
|
|
; Function Attrs: alwaysinline inlinehint uwtable
|
|
define linkonce_odr void @_Z2f4v() #1 comdat {
|
|
entry:
|
|
call void @_Z2f3v(), !dbg !15
|
|
ret void, !dbg !16
|
|
}
|
|
|
|
; Function Attrs: alwaysinline inlinehint uwtable
|
|
define linkonce_odr void @_Z2f3v() #1 comdat {
|
|
entry:
|
|
call void @_Z2f2v(), !dbg !17
|
|
call void @_Z2f2v(), !dbg !17
|
|
ret void, !dbg !18
|
|
}
|
|
|
|
; Function Attrs: alwaysinline inlinehint uwtable
|
|
define linkonce_odr void @_Z2f2v() #1 comdat {
|
|
entry:
|
|
call void @_Z2f1v(), !dbg !19
|
|
ret void, !dbg !20
|
|
}
|
|
|
|
declare void @_Z2f1v() #2
|
|
|
|
attributes #0 = { uwtable "less-precise-fpmad"="false" "no-frame-pointer-elim"="true" "no-frame-pointer-elim-non-leaf" "no-infs-fp-math"="false" "no-nans-fp-math"="false" "stack-protector-buffer-size"="8" "unsafe-fp-math"="false" "use-soft-float"="false" }
|
|
attributes #1 = { alwaysinline inlinehint uwtable "less-precise-fpmad"="false" "no-frame-pointer-elim"="true" "no-frame-pointer-elim-non-leaf" "no-infs-fp-math"="false" "no-nans-fp-math"="false" "stack-protector-buffer-size"="8" "unsafe-fp-math"="false" "use-soft-float"="false" }
|
|
attributes #2 = { "less-precise-fpmad"="false" "no-frame-pointer-elim"="true" "no-frame-pointer-elim-non-leaf" "no-infs-fp-math"="false" "no-nans-fp-math"="false" "stack-protector-buffer-size"="8" "unsafe-fp-math"="false" "use-soft-float"="false" }
|
|
|
|
!llvm.dbg.cu = !{!0}
|
|
!llvm.module.flags = !{!10, !11}
|
|
!llvm.ident = !{!12}
|
|
|
|
!0 = !{!"0x11\004\00clang version 3.7.0 (trunk 226474) (llvm/trunk 226478)\000\00\000\00\002", !1, !2, !2, !3, !2, !2} ; [ DW_TAG_compile_unit ] [/tmp/dbginfo/debug-info-duplicate-calls.cpp] [DW_LANG_C_plus_plus]
|
|
!1 = !{!"debug-info-duplicate-calls.cpp", !"/tmp/dbginfo"}
|
|
!2 = !{}
|
|
!3 = !{!4, !7, !8, !9}
|
|
!4 = !{!"0x2e\00f\00f\00\0013\000\001\000\000\00256\000\0013", !1, !5, !6, null, void ()* @_Z1fv, null, null, !2} ; [ DW_TAG_subprogram ] [line 13] [def] [f]
|
|
!5 = !{!"0x29", !1} ; [ DW_TAG_file_type ] [/tmp/dbginfo/debug-info-duplicate-calls.cpp]
|
|
!6 = !{!"0x15\00\000\000\000\000\000\000", null, null, null, !2, null, null, null} ; [ DW_TAG_subroutine_type ] [line 0, size 0, align 0, offset 0] [from ]
|
|
!7 = !{!"0x2e\00f4\00f4\00\0010\000\001\000\000\00256\000\0010", !1, !5, !6, null, void ()* @_Z2f4v, null, null, !2} ; [ DW_TAG_subprogram ] [line 10] [def] [f4]
|
|
!8 = !{!"0x2e\00f3\00f3\00\007\000\001\000\000\00256\000\007", !1, !5, !6, null, void ()* @_Z2f3v, null, null, !2} ; [ DW_TAG_subprogram ] [line 7] [def] [f3]
|
|
!9 = !{!"0x2e\00f2\00f2\00\004\000\001\000\000\00256\000\004", !1, !5, !6, null, void ()* @_Z2f2v, null, null, !2} ; [ DW_TAG_subprogram ] [line 4] [def] [f2]
|
|
!10 = !{i32 2, !"Dwarf Version", i32 4}
|
|
!11 = !{i32 2, !"Debug Info Version", i32 2}
|
|
!12 = !{!"clang version 3.7.0 (trunk 226474) (llvm/trunk 226478)"}
|
|
!13 = !MDLocation(line: 14, column: 3, scope: !4)
|
|
!14 = !MDLocation(line: 15, column: 1, scope: !4)
|
|
!15 = !MDLocation(line: 11, column: 3, scope: !7)
|
|
!16 = !MDLocation(line: 12, column: 1, scope: !7)
|
|
!17 = !MDLocation(line: 8, column: 3, scope: !8)
|
|
!18 = !MDLocation(line: 9, column: 1, scope: !8)
|
|
!19 = !MDLocation(line: 5, column: 3, scope: !9)
|
|
!20 = !MDLocation(line: 6, column: 1, scope: !9)
|