mirror of
https://github.com/RPCS3/llvm-mirror.git
synced 2024-11-23 19:23:23 +01:00
fb3abba237
Currently each Function points to a DISubprogram and DISubprogram has a scope field. For member functions the scope is a DICompositeType. DIScopes point to the DICompileUnit to facilitate type uniquing. Distinct DISubprograms (with isDefinition: true) are not part of the type hierarchy and cannot be uniqued. This change removes the subprograms list from DICompileUnit and instead adds a pointer to the owning compile unit to distinct DISubprograms. This would make it easy for ThinLTO to strip unneeded DISubprograms and their transitively referenced debug info. Motivation ---------- Materializing DISubprograms is currently the most expensive operation when doing a ThinLTO build of clang. We want the DISubprogram to be stored in a separate Bitcode block (or the same block as the function body) so we can avoid having to expensively deserialize all DISubprograms together with the global metadata. If a function has been inlined into another subprogram we need to store a reference the block containing the inlined subprogram. Attached to https://llvm.org/bugs/show_bug.cgi?id=27284 is a python script that updates LLVM IR testcases to the new format. http://reviews.llvm.org/D19034 <rdar://problem/25256815> llvm-svn: 266446
49 lines
1.9 KiB
LLVM
49 lines
1.9 KiB
LLVM
; RUN: opt -inline -S < %s | FileCheck %s
|
|
|
|
; Make sure the inliner doesn't crash when a metadata-bridged SSA operand is an
|
|
; undominated use.
|
|
;
|
|
; If we ever add a verifier check to prevent the scenario in this file, it's
|
|
; fine to delete this testcase. However, we would need a bitcode upgrade since
|
|
; such historical IR exists in practice.
|
|
|
|
define i32 @foo(i32 %i) !dbg !4 {
|
|
entry:
|
|
tail call void @llvm.dbg.value(metadata i32 %add, i64 0, metadata !8, metadata !10), !dbg !11
|
|
%add = add nsw i32 1, %i, !dbg !12
|
|
ret i32 %add, !dbg !13
|
|
}
|
|
|
|
; CHECK-LABEL: define i32 @caller(
|
|
define i32 @caller(i32 %i) {
|
|
; CHECK-NEXT: entry:
|
|
entry:
|
|
; Although the inliner shouldn't crash, it can't be expected to get the
|
|
; "correct" SSA value since its assumptions have been violated.
|
|
; CHECK-NEXT: tail call void @llvm.dbg.value(metadata ![[EMPTY:[0-9]+]],
|
|
; CHECK-NEXT: %{{.*}} = add nsw
|
|
%call = tail call i32 @foo(i32 %i)
|
|
ret i32 %call
|
|
}
|
|
|
|
declare void @llvm.dbg.value(metadata, i64, metadata, metadata)
|
|
|
|
!llvm.dbg.cu = !{!0}
|
|
!llvm.module.flags = !{!9}
|
|
|
|
!0 = distinct !DICompileUnit(language: DW_LANG_C99, file: !1, producer: "clang version 3.9.0 (trunk 265634) (llvm/trunk 265637)", isOptimized: true, runtimeVersion: 0, emissionKind: FullDebug, enums: !2)
|
|
!1 = !DIFile(filename: "t.c", directory: "/path/to/tests")
|
|
|
|
; CHECK: ![[EMPTY]] = !{}
|
|
!2 = !{}
|
|
!4 = distinct !DISubprogram(name: "foo", scope: !1, file: !1, line: 2, type: !5, isLocal: false, isDefinition: true, scopeLine: 2, flags: DIFlagPrototyped, isOptimized: true, unit: !0)
|
|
!5 = !DISubroutineType(types: !6)
|
|
!6 = !{!7, !7}
|
|
!7 = !DIBasicType(name: "int", size: 32, align: 32, encoding: DW_ATE_signed)
|
|
!8 = !DILocalVariable(name: "add", arg: 1, scope: !4, file: !1, line: 2, type: !7)
|
|
!9 = !{i32 2, !"Debug Info Version", i32 3}
|
|
!10 = !DIExpression()
|
|
!11 = !DILocation(line: 2, column: 13, scope: !4)
|
|
!12 = !DILocation(line: 2, column: 27, scope: !4)
|
|
!13 = !DILocation(line: 2, column: 18, scope: !4)
|