Verifier: Ignore CUs pulled in by ODR-uniqued types.

When more than one Module is imported into the same context, such as during
an LTO build before linking the modules, ODR type uniquing may cause types
to point to a different CU. This check does not make sense in this case.

This fixes the error reported in PR34944.

https://bugs.llvm.org/show_bug.cgi?id=34944
rdar://problem/34940685

llvm-svn: 316049

2 files changed, 200 insertions, 0 deletions

diff --git a/llvm/test/Linker/Inputs/odr-lambda-2.ll b/llvm/test/Linker/Inputs/odr-lambda-2.ll
new file mode 100644
index 00000000000..8c52dbd0d41
--- /dev/null
+++ b/llvm/test/Linker/Inputs/odr-lambda-2.ll
@@ -0,0 +1,102 @@
+; ModuleID = '/tmp/odr-lambda-2.ii'
+; generated from:
+
+; clang++ -x c++ -std=c++14 -fPIC -flto -g -fno-exceptions -fno-rtti
+; class Error {};
+; template <typename HandlerTs>
+; void handleAllErrors( HandlerTs  Handlers) {}
+; inline void consumeError(Error Err) {
+;   handleAllErrors( []() {});
+; }
+; int main(int argc, char **argv) {
+;   consumeError(Error());
+; }
+
+source_filename = "/tmp/odr-lambda-2.ii"
+target datalayout = "e-m:o-i64:64-f80:128-n8:16:32:64-S128"
+target triple = "x86_64-apple-macosx10.13.0"
+
+%class.Error = type { i8 }
+%class.anon = type { i8 }
+
+; Function Attrs: noinline norecurse nounwind optnone ssp uwtable
+define i32 @main(i32 %argc, i8** %argv) #0 !dbg !8 {
+entry:
+  %argc.addr = alloca i32, align 4
+  %argv.addr = alloca i8**, align 8
+  %agg.tmp = alloca %class.Error, align 1
+  store i32 %argc, i32* %argc.addr, align 4
+  call void @llvm.dbg.declare(metadata i32* %argc.addr, metadata !15, metadata !DIExpression()), !dbg !16
+  store i8** %argv, i8*** %argv.addr, align 8
+  call void @llvm.dbg.declare(metadata i8*** %argv.addr, metadata !17, metadata !DIExpression()), !dbg !18
+  call void @_Z12consumeError5Error(), !dbg !19
+  ret i32 0, !dbg !20
+}
+
+; Function Attrs: nounwind readnone speculatable
+declare void @llvm.dbg.declare(metadata, metadata, metadata) #1
+
+; Function Attrs: noinline nounwind optnone ssp uwtable
+define linkonce_odr void @_Z12consumeError5Error() #2 !dbg !21 {
+entry:
+  %Err = alloca %class.Error, align 1
+  %agg.tmp = alloca %class.anon, align 1
+  call void @llvm.dbg.declare(metadata %class.Error* %Err, metadata !25, metadata !DIExpression()), !dbg !26
+  call void @_Z15handleAllErrorsIZ12consumeError5ErrorEUlvE_EvT_(), !dbg !27
+  ret void, !dbg !28
+}
+
+; Function Attrs: noinline nounwind optnone ssp uwtable
+define linkonce_odr void @_Z15handleAllErrorsIZ12consumeError5ErrorEUlvE_EvT_() #2 !dbg !29 {
+entry:
+  %Handlers = alloca %class.anon, align 1
+  call void @llvm.dbg.declare(metadata %class.anon* %Handlers, metadata !35, metadata !DIExpression()), !dbg !36
+  ret void, !dbg !37
+}
+
+attributes #0 = { noinline norecurse nounwind optnone ssp uwtable }
+attributes #1 = { nounwind readnone speculatable }
+attributes #2 = { noinline nounwind optnone ssp uwtable }
+
+!llvm.dbg.cu = !{!0}
+!llvm.module.flags = !{!3, !4, !5, !6}
+!llvm.ident = !{!7}
+
+!0 = distinct !DICompileUnit(language: DW_LANG_C_plus_plus, file: !1, producer: "clang version 6.0.0 (trunk 315772) (llvm/trunk 315773)", isOptimized: true, runtimeVersion: 0, emissionKind: FullDebug, enums: !2)
+!1 = !DIFile(filename: "/tmp/odr-lambda-2.ii", directory: "/Data")
+!2 = !{}
+!3 = !{i32 2, !"Dwarf Version", i32 4}
+!4 = !{i32 2, !"Debug Info Version", i32 3}
+!5 = !{i32 1, !"wchar_size", i32 4}
+!6 = !{i32 7, !"PIC Level", i32 2}
+!7 = !{!"clang version 6.0.0 (trunk 315772) (llvm/trunk 315773)"}
+!8 = distinct !DISubprogram(name: "main", scope: !1, file: !1, line: 7, type: !9, isLocal: false, isDefinition: true, scopeLine: 7, flags: DIFlagPrototyped, isOptimized: false, unit: !0, variables: !2)
+!9 = !DISubroutineType(types: !10)
+!10 = !{!11, !11, !12}
+!11 = !DIBasicType(name: "int", size: 32, encoding: DW_ATE_signed)
+!12 = !DIDerivedType(tag: DW_TAG_pointer_type, baseType: !13, size: 64)
+!13 = !DIDerivedType(tag: DW_TAG_pointer_type, baseType: !14, size: 64)
+!14 = !DIBasicType(name: "char", size: 8, encoding: DW_ATE_signed_char)
+!15 = !DILocalVariable(name: "argc", arg: 1, scope: !8, file: !1, line: 7, type: !11)
+!16 = !DILocation(line: 7, column: 14, scope: !8)
+!17 = !DILocalVariable(name: "argv", arg: 2, scope: !8, file: !1, line: 7, type: !12)
+!18 = !DILocation(line: 7, column: 27, scope: !8)
+!19 = !DILocation(line: 8, column: 3, scope: !8)
+!20 = !DILocation(line: 9, column: 1, scope: !8)
+!21 = distinct !DISubprogram(name: "consumeError", linkageName: "_Z12consumeError5Error", scope: !1, file: !1, line: 4, type: !22, isLocal: false, isDefinition: true, scopeLine: 4, flags: DIFlagPrototyped, isOptimized: false, unit: !0, variables: !2)
+!22 = !DISubroutineType(types: !23)
+!23 = !{null, !24}
+!24 = distinct !DICompositeType(tag: DW_TAG_class_type, name: "Error", file: !1, line: 1, size: 8, elements: !2, identifier: "_ZTS5Error")
+!25 = !DILocalVariable(name: "Err", arg: 1, scope: !21, file: !1, line: 4, type: !24)
+!26 = !DILocation(line: 4, column: 32, scope: !21)
+!27 = !DILocation(line: 5, column: 3, scope: !21)
+!28 = !DILocation(line: 6, column: 1, scope: !21)
+!29 = distinct !DISubprogram(name: "handleAllErrors<(lambda at /tmp/odr-lambda-2.ii:5:20)>", linkageName: "_Z15handleAllErrorsIZ12consumeError5ErrorEUlvE_EvT_", scope: !1, file: !1, line: 3, type: !30, isLocal: false, isDefinition: true, scopeLine: 3, flags: DIFlagPrototyped, isOptimized: false, unit: !0, templateParams: !33, variables: !2)
+!30 = !DISubroutineType(types: !31)
+!31 = !{null, !32}
+!32 = distinct !DICompositeType(tag: DW_TAG_class_type, scope: !21, file: !1, line: 5, size: 8, elements: !2, identifier: "_ZTSZ12consumeError5ErrorEUlvE_")
+!33 = !{!34}
+!34 = !DITemplateTypeParameter(name: "HandlerTs", type: !32)
+!35 = !DILocalVariable(name: "Handlers", arg: 1, scope: !29, file: !1, line: 3, type: !32)
+!36 = !DILocation(line: 3, column: 34, scope: !29)
+!37 = !DILocation(line: 3, column: 45, scope: !29)
diff --git a/llvm/test/Linker/odr-lambda-1.ll b/llvm/test/Linker/odr-lambda-1.ll
new file mode 100644
index 00000000000..f0eb6542704
--- /dev/null
+++ b/llvm/test/Linker/odr-lambda-1.ll
@@ -0,0 +1,98 @@
+; ModuleID = '/tmp/odr-lambda-1.ii'
+
+; RUN: llvm-link %s %p/Inputs/odr-lambda-2.ll -S -o - 2>&1 | FileCheck %s
+
+; When materializing the ODR-uniqued types they may be resolved to types from a
+; previously loaded module. Don't treat this as an error.
+; CHECK-NOT: ignoring invalid debug info
+; CHECK: !llvm.dbg.cu = !{!{{[0-9]+}}, !{{[0-9]+}}}
+; CHECK: distinct !DICompositeType(tag: DW_TAG_class_type, {{.*}}identifier: "_ZTSZ12consumeError5ErrorEUlvE_")
+; CHECK-NOT: identifier: "_ZTSZ12consumeError5ErrorEUlvE_"
+
+
+; generated from:
+; clang++ -x c++ -std=c++14 -fPIC -flto -g -fno-exceptions -fno-rtti
+; class Error {};
+; template <typename HandlerTs>
+; void handleAllErrors(HandlerTs Handlers) {}
+; inline void consumeError(Error Err) {
+;   handleAllErrors( []() {});
+; }
+; void ArchiveMemberHeader() 
+; {
+;   consumeError(Error());
+; }
+
+source_filename = "/tmp/odr-lambda-1.ii"
+target datalayout = "e-m:o-i64:64-f80:128-n8:16:32:64-S128"
+target triple = "x86_64-apple-macosx10.13.0"
+
+%class.Error = type { i8 }
+%class.anon = type { i8 }
+
+; Function Attrs: noinline nounwind optnone ssp uwtable
+define void @_Z19ArchiveMemberHeaderv() #0 !dbg !8 {
+entry:
+  %agg.tmp = alloca %class.Error, align 1
+  call void @_Z12consumeError5Error(), !dbg !11
+  ret void, !dbg !12
+}
+
+; Function Attrs: noinline nounwind optnone ssp uwtable
+define linkonce_odr void @_Z12consumeError5Error() #0 !dbg !13 {
+entry:
+  %Err = alloca %class.Error, align 1
+  %agg.tmp = alloca %class.anon, align 1
+  call void @llvm.dbg.declare(metadata %class.Error* %Err, metadata !17, metadata !DIExpression()), !dbg !18
+  call void @_Z15handleAllErrorsIZ12consumeError5ErrorEUlvE_EvT_(), !dbg !19
+  ret void, !dbg !20
+}
+
+; Function Attrs: nounwind readnone speculatable
+declare void @llvm.dbg.declare(metadata, metadata, metadata) #1
+
+; Function Attrs: noinline nounwind optnone ssp uwtable
+define linkonce_odr void @_Z15handleAllErrorsIZ12consumeError5ErrorEUlvE_EvT_() #0 !dbg !21 {
+entry:
+  %Handlers = alloca %class.anon, align 1
+  call void @llvm.dbg.declare(metadata %class.anon* %Handlers, metadata !27, metadata !DIExpression()), !dbg !28
+  ret void, !dbg !29
+}
+
+attributes #0 = { noinline nounwind optnone ssp uwtable }
+attributes #1 = { nounwind readnone speculatable }
+
+!llvm.dbg.cu = !{!0}
+!llvm.module.flags = !{!3, !4, !5, !6}
+!llvm.ident = !{!7}
+
+!0 = distinct !DICompileUnit(language: DW_LANG_C_plus_plus, file: !1, producer: "clang version 6.0.0 (trunk 315772) (llvm/trunk 315773)", isOptimized: true, runtimeVersion: 0, emissionKind: FullDebug, enums: !2)
+!1 = !DIFile(filename: "/tmp/odr-lambda-1.ii", directory: "/Data")
+!2 = !{}
+!3 = !{i32 2, !"Dwarf Version", i32 4}
+!4 = !{i32 2, !"Debug Info Version", i32 3}
+!5 = !{i32 1, !"wchar_size", i32 4}
+!6 = !{i32 7, !"PIC Level", i32 2}
+!7 = !{!"clang version 6.0.0 (trunk 315772) (llvm/trunk 315773)"}
+!8 = distinct !DISubprogram(name: "ArchiveMemberHeader", linkageName: "_Z19ArchiveMemberHeaderv", scope: !1, file: !1, line: 7, type: !9, isLocal: false, isDefinition: true, scopeLine: 8, flags: DIFlagPrototyped, isOptimized: false, unit: !0, variables: !2)
+!9 = !DISubroutineType(types: !10)
+!10 = !{null}
+!11 = !DILocation(line: 9, column: 3, scope: !8)
+!12 = !DILocation(line: 10, column: 1, scope: !8)
+!13 = distinct !DISubprogram(name: "consumeError", linkageName: "_Z12consumeError5Error", scope: !1, file: !1, line: 4, type: !14, isLocal: false, isDefinition: true, scopeLine: 4, flags: DIFlagPrototyped, isOptimized: false, unit: !0, variables: !2)
+!14 = !DISubroutineType(types: !15)
+!15 = !{null, !16}
+!16 = distinct !DICompositeType(tag: DW_TAG_class_type, name: "Error", file: !1, line: 1, size: 8, elements: !2, identifier: "_ZTS5Error")
+!17 = !DILocalVariable(name: "Err", arg: 1, scope: !13, file: !1, line: 4, type: !16)
+!18 = !DILocation(line: 4, column: 32, scope: !13)
+!19 = !DILocation(line: 5, column: 3, scope: !13)
+!20 = !DILocation(line: 6, column: 1, scope: !13)
+!21 = distinct !DISubprogram(name: "handleAllErrors<(lambda at /tmp/odr-lambda-1.ii:5:20)>", linkageName: "_Z15handleAllErrorsIZ12consumeError5ErrorEUlvE_EvT_", scope: !1, file: !1, line: 3, type: !22, isLocal: false, isDefinition: true, scopeLine: 3, flags: DIFlagPrototyped, isOptimized: false, unit: !0, templateParams: !25, variables: !2)
+!22 = !DISubroutineType(types: !23)
+!23 = !{null, !24}
+!24 = distinct !DICompositeType(tag: DW_TAG_class_type, scope: !13, file: !1, line: 5, size: 8, elements: !2, identifier: "_ZTSZ12consumeError5ErrorEUlvE_")
+!25 = !{!26}
+!26 = !DITemplateTypeParameter(name: "HandlerTs", type: !24)
+!27 = !DILocalVariable(name: "Handlers", arg: 1, scope: !21, file: !1, line: 3, type: !24)
+!28 = !DILocation(line: 3, column: 32, scope: !21)
+!29 = !DILocation(line: 3, column: 43, scope: !21)