<div dir="ltr">Do we need a test for the verifier to demonstrate it does catch this?</div><br><div class="gmail_quote"><div dir="ltr">On Wed, Oct 5, 2016 at 2:40 PM Adrian Prantl via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Author: adrian<br class="gmail_msg">
Date: Wed Oct  5 16:31:19 2016<br class="gmail_msg">
New Revision: 283390<br class="gmail_msg">
<br class="gmail_msg">
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=283390&view=rev" rel="noreferrer" class="gmail_msg" target="_blank">http://llvm.org/viewvc/llvm-project?rev=283390&view=rev</a><br class="gmail_msg">
Log:<br class="gmail_msg">
Verifier: Reject any unknown named MD nodes in the llvm.dbg namespace.<br class="gmail_msg">
<br class="gmail_msg">
This came out of a discussion in <a href="https://reviews.llvm.org/D25285" rel="noreferrer" class="gmail_msg" target="_blank">https://reviews.llvm.org/D25285</a>.<br class="gmail_msg">
<br class="gmail_msg">
There used to be various other llvm.dbg.* nodes, but we don't support<br class="gmail_msg">
upgrading them and we want to reserve the namespace for future uses.<br class="gmail_msg">
<br class="gmail_msg">
This also removes an entirely obsolete and bitrotted testcase for PR7662.<br class="gmail_msg">
<br class="gmail_msg">
Removed:<br class="gmail_msg">
    llvm/trunk/test/DebugInfo/Generic/2010-07-19-Crash.ll<br class="gmail_msg">
Modified:<br class="gmail_msg">
    llvm/trunk/lib/IR/Verifier.cpp<br class="gmail_msg">
    llvm/trunk/test/CodeGen/X86/2010-06-01-DeadArg-DbgInfo.ll<br class="gmail_msg">
<br class="gmail_msg">
Modified: llvm/trunk/lib/IR/Verifier.cpp<br class="gmail_msg">
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/IR/Verifier.cpp?rev=283390&r1=283389&r2=283390&view=diff" rel="noreferrer" class="gmail_msg" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/IR/Verifier.cpp?rev=283390&r1=283389&r2=283390&view=diff</a><br class="gmail_msg">
==============================================================================<br class="gmail_msg">
--- llvm/trunk/lib/IR/Verifier.cpp (original)<br class="gmail_msg">
+++ llvm/trunk/lib/IR/Verifier.cpp Wed Oct  5 16:31:19 2016<br class="gmail_msg">
@@ -698,10 +698,15 @@ void Verifier::visitGlobalAlias(const Gl<br class="gmail_msg">
 }<br class="gmail_msg">
<br class="gmail_msg">
 void Verifier::visitNamedMDNode(const NamedMDNode &NMD) {<br class="gmail_msg">
+  // There used to be various other llvm.dbg.* nodes, but we don't support<br class="gmail_msg">
+  // upgrading them and we want to reserve the namespace for future uses.<br class="gmail_msg">
+  if (NMD.getName().startswith("llvm.dbg."))<br class="gmail_msg">
+    AssertDI(NMD.getName() == "<a href="http://llvm.dbg.cu" rel="noreferrer" class="gmail_msg" target="_blank">llvm.dbg.cu</a>",<br class="gmail_msg">
+             "unrecognized named metadata node in the llvm.dbg namespace",<br class="gmail_msg">
+             &NMD);<br class="gmail_msg">
   for (const MDNode *MD : NMD.operands()) {<br class="gmail_msg">
-    if (NMD.getName() == "<a href="http://llvm.dbg.cu" rel="noreferrer" class="gmail_msg" target="_blank">llvm.dbg.cu</a>") {<br class="gmail_msg">
+    if (NMD.getName() == "<a href="http://llvm.dbg.cu" rel="noreferrer" class="gmail_msg" target="_blank">llvm.dbg.cu</a>")<br class="gmail_msg">
       AssertDI(MD && isa<DICompileUnit>(MD), "invalid compile unit", &NMD, MD);<br class="gmail_msg">
-    }<br class="gmail_msg">
<br class="gmail_msg">
     if (!MD)<br class="gmail_msg">
       continue;<br class="gmail_msg">
<br class="gmail_msg">
Modified: llvm/trunk/test/CodeGen/X86/2010-06-01-DeadArg-DbgInfo.ll<br class="gmail_msg">
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/X86/2010-06-01-DeadArg-DbgInfo.ll?rev=283390&r1=283389&r2=283390&view=diff" rel="noreferrer" class="gmail_msg" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/X86/2010-06-01-DeadArg-DbgInfo.ll?rev=283390&r1=283389&r2=283390&view=diff</a><br class="gmail_msg">
==============================================================================<br class="gmail_msg">
--- llvm/trunk/test/CodeGen/X86/2010-06-01-DeadArg-DbgInfo.ll (original)<br class="gmail_msg">
+++ llvm/trunk/test/CodeGen/X86/2010-06-01-DeadArg-DbgInfo.ll Wed Oct  5 16:31:19 2016<br class="gmail_msg">
@@ -3,57 +3,49 @@<br class="gmail_msg">
 ; Test to check that unused argument 'this' is not undefined in debug info.<br class="gmail_msg">
<br class="gmail_msg">
 target triple = "x86_64-apple-darwin10.2"<br class="gmail_msg">
+<br class="gmail_msg">
 %struct.foo = type { i32 }<br class="gmail_msg">
<br class="gmail_msg">
-@llvm.used = appending global [1 x i8*] [i8* bitcast (i32 (%struct.foo*, i32)* @_ZN3foo3bazEi to i8*)], section "llvm.metadata" ; <[1 x i8*]*> [#uses=0]<br class="gmail_msg">
+@llvm.used = appending global [1 x i8*] [i8* bitcast (i32 (%struct.foo*, i32)* @_ZN3foo3bazEi to i8*)], section "llvm.metadata"<br class="gmail_msg">
<br class="gmail_msg">
-define i32 @_ZN3foo3bazEi(%struct.foo* nocapture %this, i32 %x) nounwind readnone optsize noinline ssp align 2 !dbg !8 {<br class="gmail_msg">
-;CHECK: DEBUG_VALUE: baz:this <- %RDI{{$}}<br class="gmail_msg">
-entry:<br class="gmail_msg">
-  tail call void @llvm.dbg.value(metadata %struct.foo* %this, i64 0, metadata !15, metadata !DIExpression()), !dbg !DILocation(scope: !8)<br class="gmail_msg">
-  tail call void @llvm.dbg.value(metadata i32 %x, i64 0, metadata !16, metadata !DIExpression()), !dbg !DILocation(scope: !8)<br class="gmail_msg">
-  %0 = mul nsw i32 %x, 7, !dbg !29                ; <i32> [#uses=1]<br class="gmail_msg">
-  %1 = add nsw i32 %0, 1, !dbg !29                ; <i32> [#uses=1]<br class="gmail_msg">
-  ret i32 %1, !dbg !29<br class="gmail_msg">
+; Function Attrs: noinline nounwind optsize readnone ssp<br class="gmail_msg">
+define i32 @_ZN3foo3bazEi(%struct.foo* nocapture %this, i32 %x) #0 align 2 !dbg !4 {<br class="gmail_msg">
+entry:<br class="gmail_msg">
+  ; CHECK: DEBUG_VALUE: baz:this <- %RDI{{$}}<br class="gmail_msg">
+  tail call void @llvm.dbg.value(metadata %struct.foo* %this, i64 0, metadata !13, metadata !16), !dbg !17<br class="gmail_msg">
+  tail call void @llvm.dbg.value(metadata i32 %x, i64 0, metadata !18, metadata !16), !dbg !17<br class="gmail_msg">
+  %0 = mul nsw i32 %x, 7, !dbg !19<br class="gmail_msg">
+  %1 = add nsw i32 %0, 1, !dbg !19<br class="gmail_msg">
+  ret i32 %1, !dbg !19<br class="gmail_msg">
 }<br class="gmail_msg">
<br class="gmail_msg">
-declare void @llvm.dbg.value(metadata, i64, metadata, metadata) nounwind readnone<br class="gmail_msg">
+; Function Attrs: nounwind readnone<br class="gmail_msg">
+declare void @llvm.dbg.value(metadata, i64, metadata, metadata) #1<br class="gmail_msg">
+<br class="gmail_msg">
+attributes #0 = { noinline nounwind optsize readnone ssp }<br class="gmail_msg">
+attributes #1 = { nounwind readnone }<br class="gmail_msg">
+<br class="gmail_msg">
+!<a href="http://llvm.dbg.cu" rel="noreferrer" class="gmail_msg" target="_blank">llvm.dbg.cu</a> = !{!0}<br class="gmail_msg">
+!llvm.module.flags = !{!3}<br class="gmail_msg">
<br class="gmail_msg">
-!<a href="http://llvm.dbg.cu" rel="noreferrer" class="gmail_msg" target="_blank">llvm.dbg.cu</a> = !{!4}<br class="gmail_msg">
-!llvm.module.flags = !{!34}<br class="gmail_msg">
-!<a href="http://llvm.dbg.lv" rel="noreferrer" class="gmail_msg" target="_blank">llvm.dbg.lv</a> = !{!0, !14, !15, !16, !17, !24, !25, !28}<br class="gmail_msg">
-<br class="gmail_msg">
-!0 = !DILocalVariable(name: "this", line: 11, arg: 1, scope: !1, file: !3, type: !12)<br class="gmail_msg">
-!1 = distinct !DISubprogram(name: "bar", linkageName: "_ZN3foo3barEi", line: 11, isLocal: false, isDefinition: true, virtualIndex: 6, isOptimized: true, unit: !4, scopeLine: 11, file: !31, scope: !2, type: !9)<br class="gmail_msg">
-!2 = !DICompositeType(tag: DW_TAG_structure_type, name: "foo", line: 3, size: 32, align: 32, file: !31, scope: !3, elements: !5)<br class="gmail_msg">
-!3 = !DIFile(filename: "foo.cp", directory: "/tmp/")<br class="gmail_msg">
-!4 = distinct !DICompileUnit(language: DW_LANG_C_plus_plus, producer: "4.2.1 LLVM build", isOptimized: true, emissionKind: FullDebug, file: !31, enums: !32, retainedTypes: !32)<br class="gmail_msg">
-!5 = !{!6, !1, !8}<br class="gmail_msg">
-!6 = !DIDerivedType(tag: DW_TAG_member, name: "y", line: 8, size: 32, align: 32, file: !31, scope: !2, baseType: !7)<br class="gmail_msg">
-!7 = !DIBasicType(tag: DW_TAG_base_type, name: "int", size: 32, align: 32, encoding: DW_ATE_signed)<br class="gmail_msg">
-!8 = distinct !DISubprogram(name: "baz", linkageName: "_ZN3foo3bazEi", line: 15, isLocal: false, isDefinition: true, virtualIndex: 6, isOptimized: true, unit: !4, scopeLine: 15, file: !31, scope: !2, type: !9)<br class="gmail_msg">
-!9 = !DISubroutineType(types: !10)<br class="gmail_msg">
-!10 = !{!7, !11, !7}<br class="gmail_msg">
-!11 = !DIDerivedType(tag: DW_TAG_pointer_type, size: 64, align: 64, flags: DIFlagArtificial, file: !31, scope: !3, baseType: !2)<br class="gmail_msg">
-!12 = !DIDerivedType(tag: DW_TAG_const_type, size: 64, align: 64, flags: DIFlagArtificial, file: !31, scope: !3, baseType: !13)<br class="gmail_msg">
-!13 = !DIDerivedType(tag: DW_TAG_pointer_type, size: 64, align: 64, file: !31, scope: !3, baseType: !2)<br class="gmail_msg">
-!14 = !DILocalVariable(name: "x", line: 11, arg: 2, scope: !1, file: !3, type: !7)<br class="gmail_msg">
-!15 = !DILocalVariable(name: "this", line: 15, arg: 1, scope: !8, file: !3, type: !12)<br class="gmail_msg">
-!16 = !DILocalVariable(name: "x", line: 15, arg: 2, scope: !8, file: !3, type: !7)<br class="gmail_msg">
-!17 = !DILocalVariable(name: "argc", line: 19, arg: 1, scope: !18, file: !3, type: !7)<br class="gmail_msg">
-!18 = distinct !DISubprogram(name: "main", linkageName: "main", line: 19, isLocal: false, isDefinition: true, virtualIndex: 6, isOptimized: true, unit: !4, scopeLine: 19, file: !31, scope: !3, type: !19)<br class="gmail_msg">
-!19 = !DISubroutineType(types: !20)<br class="gmail_msg">
-!20 = !{!7, !7, !21}<br class="gmail_msg">
-!21 = !DIDerivedType(tag: DW_TAG_pointer_type, size: 64, align: 64, file: !31, scope: !3, baseType: !22)<br class="gmail_msg">
-!22 = !DIDerivedType(tag: DW_TAG_pointer_type, size: 64, align: 64, file: !31, scope: !3, baseType: !23)<br class="gmail_msg">
-!23 = !DIBasicType(tag: DW_TAG_base_type, name: "char", size: 8, align: 8, encoding: DW_ATE_signed_char)<br class="gmail_msg">
-!24 = !DILocalVariable(name: "argv", line: 19, arg: 2, scope: !18, file: !3, type: !21)<br class="gmail_msg">
-!25 = !DILocalVariable(name: "a", line: 20, scope: !26, file: !3, type: !2)<br class="gmail_msg">
-!26 = distinct !DILexicalBlock(line: 19, column: 0, file: !31, scope: !27)<br class="gmail_msg">
-!27 = distinct !DILexicalBlock(line: 19, column: 0, file: !31, scope: !18)<br class="gmail_msg">
-!28 = !DILocalVariable(name: "b", line: 21, scope: !26, file: !3, type: !7)<br class="gmail_msg">
-!29 = !DILocation(line: 16, scope: !30)<br class="gmail_msg">
-!30 = distinct !DILexicalBlock(line: 15, column: 0, file: !31, scope: !8)<br class="gmail_msg">
-!31 = !DIFile(filename: "foo.cp", directory: "/tmp/")<br class="gmail_msg">
-!32 = !{}<br class="gmail_msg">
-!34 = !{i32 1, !"Debug Info Version", i32 3}<br class="gmail_msg">
+!0 = distinct !DICompileUnit(language: DW_LANG_C_plus_plus, file: !1, producer: "4.2.1 LLVM build", isOptimized: true, runtimeVersion: 0, emissionKind: FullDebug, enums: !2, retainedTypes: !2)<br class="gmail_msg">
+!1 = !DIFile(filename: "foo.cp", directory: "/tmp/")<br class="gmail_msg">
+!2 = !{}<br class="gmail_msg">
+!3 = !{i32 1, !"Debug Info Version", i32 3}<br class="gmail_msg">
+!4 = distinct !DISubprogram(name: "baz", linkageName: "_ZN3foo3bazEi", scope: !5, file: !1, line: 15, type: !10, isLocal: false, isDefinition: true, scopeLine: 15, virtualIndex: 6, isOptimized: true, unit: !0)<br class="gmail_msg">
+!5 = !DICompositeType(tag: DW_TAG_structure_type, name: "foo", scope: !1, file: !1, line: 3, size: 32, align: 32, elements: !6)<br class="gmail_msg">
+!6 = !{!7, !9, !4}<br class="gmail_msg">
+!7 = !DIDerivedType(tag: DW_TAG_member, name: "y", scope: !5, file: !1, line: 8, baseType: !8, size: 32, align: 32)<br class="gmail_msg">
+!8 = !DIBasicType(name: "int", size: 32, align: 32, encoding: DW_ATE_signed)<br class="gmail_msg">
+!9 = distinct !DISubprogram(name: "bar", linkageName: "_ZN3foo3barEi", scope: !5, file: !1, line: 11, type: !10, isLocal: false, isDefinition: true, scopeLine: 11, virtualIndex: 6, isOptimized: true, unit: !0)<br class="gmail_msg">
+!10 = !DISubroutineType(types: !11)<br class="gmail_msg">
+!11 = !{!8, !12, !8}<br class="gmail_msg">
+!12 = !DIDerivedType(tag: DW_TAG_pointer_type, scope: !1, file: !1, baseType: !5, size: 64, align: 64, flags: DIFlagArtificial)<br class="gmail_msg">
+!13 = !DILocalVariable(name: "this", arg: 1, scope: !4, file: !1, line: 15, type: !14)<br class="gmail_msg">
+!14 = !DIDerivedType(tag: DW_TAG_const_type, scope: !1, file: !1, baseType: !15, size: 64, align: 64, flags: DIFlagArtificial)<br class="gmail_msg">
+!15 = !DIDerivedType(tag: DW_TAG_pointer_type, scope: !1, file: !1, baseType: !5, size: 64, align: 64)<br class="gmail_msg">
+!16 = !DIExpression()<br class="gmail_msg">
+!17 = !DILocation(line: 0, scope: !4)<br class="gmail_msg">
+!18 = !DILocalVariable(name: "x", arg: 2, scope: !4, file: !1, line: 15, type: !8)<br class="gmail_msg">
+!19 = !DILocation(line: 16, scope: !20)<br class="gmail_msg">
+!20 = distinct !DILexicalBlock(scope: !4, file: !1, line: 15)<br class="gmail_msg">
<br class="gmail_msg">
Removed: llvm/trunk/test/DebugInfo/Generic/2010-07-19-Crash.ll<br class="gmail_msg">
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/DebugInfo/Generic/2010-07-19-Crash.ll?rev=283389&view=auto" rel="noreferrer" class="gmail_msg" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/test/DebugInfo/Generic/2010-07-19-Crash.ll?rev=283389&view=auto</a><br class="gmail_msg">
==============================================================================<br class="gmail_msg">
--- llvm/trunk/test/DebugInfo/Generic/2010-07-19-Crash.ll (original)<br class="gmail_msg">
+++ llvm/trunk/test/DebugInfo/Generic/2010-07-19-Crash.ll (removed)<br class="gmail_msg">
@@ -1,28 +0,0 @@<br class="gmail_msg">
-; RUN: llc -o /dev/null < %s<br class="gmail_msg">
-; PR7662<br class="gmail_msg">
-; Do not add variables to !11 because it is a declaration entry.<br class="gmail_msg">
-<br class="gmail_msg">
-define i32 @bar() nounwind readnone ssp !dbg !0 {<br class="gmail_msg">
-entry:<br class="gmail_msg">
-  ret i32 42, !dbg !9<br class="gmail_msg">
-}<br class="gmail_msg">
-<br class="gmail_msg">
-!<a href="http://llvm.dbg.cu" rel="noreferrer" class="gmail_msg" target="_blank">llvm.dbg.cu</a> = !{!2}<br class="gmail_msg">
-!llvm.module.flags = !{!15}<br class="gmail_msg">
-!llvm.dbg.lv.foo = !{!7}<br class="gmail_msg">
-<br class="gmail_msg">
-!0 = distinct !DISubprogram(name: "bar", linkageName: "bar", line: 3, isLocal: false, isDefinition: true, virtualIndex: 6, isOptimized: true, unit: !2, file: !12, scope: !1, type: !3)<br class="gmail_msg">
-!1 = !DIFile(filename: "one.c", directory: "/private/tmp")<br class="gmail_msg">
-!2 = distinct !DICompileUnit(language: DW_LANG_C99, producer: "clang 2.8", isOptimized: true, emissionKind: FullDebug, file: !12, enums: !14, retainedTypes: !14)<br class="gmail_msg">
-!3 = !DISubroutineType(types: !4)<br class="gmail_msg">
-!4 = !{!5}<br class="gmail_msg">
-!5 = !DIBasicType(tag: DW_TAG_base_type, name: "int", size: 32, align: 32, encoding: DW_ATE_signed)<br class="gmail_msg">
-!6 = distinct !DISubprogram(name: "foo", linkageName: "foo", line: 7, isLocal: true, isDefinition: true, virtualIndex: 6, isOptimized: true, unit: !2, file: !12, scope: !1, type: !3, declaration: !11)<br class="gmail_msg">
-!7 = !DILocalVariable(name: "one", line: 8, scope: !8, file: !1, type: !5)<br class="gmail_msg">
-!8 = distinct !DILexicalBlock(line: 7, column: 18, file: !12, scope: !6)<br class="gmail_msg">
-!9 = !DILocation(line: 4, column: 3, scope: !10)<br class="gmail_msg">
-!10 = distinct !DILexicalBlock(line: 3, column: 11, file: !12, scope: !0)<br class="gmail_msg">
-!11 = !DISubprogram(name: "foo", linkageName: "foo", line: 7, isLocal: true, isDefinition: false, virtualIndex: 6, isOptimized: true, file: !12, scope: !1, type: !3)<br class="gmail_msg">
-!12 = !DIFile(filename: "one.c", directory: "/private/tmp")<br class="gmail_msg">
-!14 = !{}<br class="gmail_msg">
-!15 = !{i32 1, !"Debug Info Version", i32 3}<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
_______________________________________________<br class="gmail_msg">
llvm-commits mailing list<br class="gmail_msg">
<a href="mailto:llvm-commits@lists.llvm.org" class="gmail_msg" target="_blank">llvm-commits@lists.llvm.org</a><br class="gmail_msg">
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" class="gmail_msg" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br class="gmail_msg">
</blockquote></div>