diff options
-rw-r--r-- | lib/IR/Verifier.cpp | 6 | ||||
-rw-r--r-- | test/Linker/Inputs/odr-lambda-2.ll | 102 | ||||
-rw-r--r-- | test/Linker/odr-lambda-1.ll | 98 |
3 files changed, 206 insertions, 0 deletions
diff --git a/lib/IR/Verifier.cpp b/lib/IR/Verifier.cpp index 7c6e4585b9e..ceaead99531 100644 --- a/lib/IR/Verifier.cpp +++ b/lib/IR/Verifier.cpp @@ -45,6 +45,7 @@ //===----------------------------------------------------------------------===// #include "llvm/IR/Verifier.h" +#include "LLVMContextImpl.h" #include "llvm/ADT/APFloat.h" #include "llvm/ADT/APInt.h" #include "llvm/ADT/ArrayRef.h" @@ -4593,6 +4594,11 @@ void Verifier::verifyFnArgs(const DbgInfoIntrinsic &I) { } void Verifier::verifyCompileUnits() { + // 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. + if (M.getContext().pImpl->OwnedModules.size() > 1) + return; auto *CUs = M.getNamedMetadata("llvm.dbg.cu"); SmallPtrSet<const Metadata *, 2> Listed; if (CUs) diff --git a/test/Linker/Inputs/odr-lambda-2.ll b/test/Linker/Inputs/odr-lambda-2.ll new file mode 100644 index 00000000000..8c52dbd0d41 --- /dev/null +++ b/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/test/Linker/odr-lambda-1.ll b/test/Linker/odr-lambda-1.ll new file mode 100644 index 00000000000..f0eb6542704 --- /dev/null +++ b/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) |