[llvm] r311805 - [Verifier] Diagnose invalid DIType references instead of crashing.

[Davide Italiano via llvm-commits]

Author: davide
Date: Fri Aug 25 15:08:15 2017
New Revision: 311805

URL: http://llvm.org/viewvc/llvm-project?rev=311805&view=rev
Log:
[Verifier] Diagnose invalid DIType references instead of crashing.

Fixes PR34325.

Added:
    llvm/trunk/test/Verifier/pr34325.ll
Modified:
    llvm/trunk/lib/IR/Verifier.cpp

Modified: llvm/trunk/lib/IR/Verifier.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/IR/Verifier.cpp?rev=311805&r1=311804&r2=311805&view=diff
==============================================================================
--- llvm/trunk/lib/IR/Verifier.cpp (original)
+++ llvm/trunk/lib/IR/Verifier.cpp Fri Aug 25 15:08:15 2017
@@ -1154,6 +1154,7 @@ void Verifier::visitDIGlobalVariable(con
 
   AssertDI(N.getTag() == dwarf::DW_TAG_variable, "invalid tag", &N);
   AssertDI(!N.getName().empty(), "missing global variable name", &N);
+  AssertDI(isType(N.getRawType()), "invalid type ref", &N, N.getRawType());
   AssertDI(N.getType(), "missing global variable type", &N);
   if (auto *Member = N.getRawStaticDataMemberDeclaration()) {
     AssertDI(isa<DIDerivedType>(Member),

Added: llvm/trunk/test/Verifier/pr34325.ll
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Verifier/pr34325.ll?rev=311805&view=auto
==============================================================================
--- llvm/trunk/test/Verifier/pr34325.ll (added)
+++ llvm/trunk/test/Verifier/pr34325.ll Fri Aug 25 15:08:15 2017
@@ -0,0 +1,18 @@
+; RUN: not opt -verify %s 2>&1 | FileCheck %s
+
+; CHECK: invalid type ref
+
+ at bar = global i64 0, align 8, !dbg !0
+
+!llvm.dbg.cu = !{!2}
+!llvm.module.flags = !{!8}
+
+!0 = !DIGlobalVariableExpression(var: !1)
+!1 = !DIGlobalVariable(name: "c", scope: null, isLocal: false, isDefinition: true, type: !6)
+!2 = distinct !DICompileUnit(language: DW_LANG_C_plus_plus, file: !3, isOptimized: false, runtimeVersion: 0, emissionKind: NoDebug, retainedTypes: !4, globals: !7)
+!3 = !DIFile(filename: "a", directory: "")
+!4 = !{!5}
+!5 = distinct !DICompositeType(tag: DW_TAG_class_type, scope: !6, file: !3, identifier: "patatino")
+!6 = distinct !DISubprogram(name: "b", scope: null, isLocal: false, isDefinition: true, isOptimized: false, unit: !2)
+!7 = !{!0}
+!8 = !{i32 2, !"Debug Info Version", i32 3}