summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDavide Italiano <davide@freebsd.org>2016-04-08 00:01:32 +0000
committerDavide Italiano <davide@freebsd.org>2016-04-08 00:01:32 +0000
commit08f8f21b91e4a805ea71f48c1b04795a14da0f75 (patch)
tree5db2769995aa8934d92dde7be79de99ddf89c871
parent25c36fd61bbbc85632273d91ecb473fd4c472d0f (diff)
downloadbcm5719-llvm-08f8f21b91e4a805ea71f48c1b04795a14da0f75.tar.gz
bcm5719-llvm-08f8f21b91e4a805ea71f48c1b04795a14da0f75.zip
[IR/Verifier] Fix (yet another) crash.
We need to check that if we reference a retainedType from DICompileUnit we're actually referencing a DICompositeType. llvm-svn: 265752
-rw-r--r--llvm/lib/IR/Verifier.cpp22
-rw-r--r--llvm/test/Verifier/dbg-invalid-retaintypes.ll10
2 files changed, 24 insertions, 8 deletions
diff --git a/llvm/lib/IR/Verifier.cpp b/llvm/lib/IR/Verifier.cpp
index fc9cc7d27d5..abb0e9b32a9 100644
--- a/llvm/lib/IR/Verifier.cpp
+++ b/llvm/lib/IR/Verifier.cpp
@@ -4380,14 +4380,20 @@ void Verifier::verifyTypeRefs() {
// Visit all the compile units again to map the type references.
SmallDenseMap<const MDString *, const DIType *, 32> TypeRefs;
- for (auto *MD : CUs->operands())
- if (auto *CU = dyn_cast<DICompileUnit>(MD))
- for (DIType *Op : CU->getRetainedTypes())
- if (auto *T = dyn_cast_or_null<DICompositeType>(Op))
- if (auto *S = T->getRawIdentifier()) {
- UnresolvedTypeRefs.erase(S);
- TypeRefs.insert(std::make_pair(S, T));
- }
+ for (auto *MD : CUs->operands()) {
+ auto *CU = dyn_cast<DICompileUnit>(MD);
+ if (!CU)
+ continue;
+ auto *Array = CU->getRawRetainedTypes();
+ if (!Array || !isa<MDTuple>(Array))
+ continue;
+ for (DIType *Op : CU->getRetainedTypes())
+ if (auto *T = dyn_cast_or_null<DICompositeType>(Op))
+ if (auto *S = T->getRawIdentifier()) {
+ UnresolvedTypeRefs.erase(S);
+ TypeRefs.insert(std::make_pair(S, T));
+ }
+ }
// Verify debug info intrinsic bit piece expressions. This needs a second
// pass through the intructions, since we haven't built TypeRefs yet when
diff --git a/llvm/test/Verifier/dbg-invalid-retaintypes.ll b/llvm/test/Verifier/dbg-invalid-retaintypes.ll
new file mode 100644
index 00000000000..fa14ae82a8b
--- /dev/null
+++ b/llvm/test/Verifier/dbg-invalid-retaintypes.ll
@@ -0,0 +1,10 @@
+; RUN: not llvm-as -disable-output <%s 2>&1 | FileCheck %s
+; CHECK: assembly parsed, but does not verify
+
+!llvm.dbg.cu = !{!0}
+!llvm.module.flags = !{!3}
+
+!0 = distinct !DICompileUnit(language: DW_LANG_C99, producer: "clang version 3.0 (trunk 131941)", isOptimized: true, emissionKind: FullDebug, file: !2, retainedTypes: !1)
+!1 = distinct !DISubprogram(name: "main", line: 1, isLocal: false, isDefinition: true, virtualIndex: 6, flags: DIFlagPrototyped, isOptimized: false, scopeLine: 1, scope: !2)
+!2 = !DIFile(filename: "/davide/test", directory: "/")
+!3 = !{i32 1, !"Debug Info Version", i32 3}
OpenPOWER on IntegriCloud