From 3807217d028dd00ba3d7c3476f226c9d12d52007 Mon Sep 17 00:00:00 2001 From: Adrian Prantl Date: Fri, 24 Aug 2018 21:01:58 +0000 Subject: [PATCH] Verifier: verify that a DILocation's scope is a DILocalScope. This fixes an assertion failure(!) in the Verifier. rdar://problem/43687474 llvm-svn: 340653 --- llvm/lib/IR/Verifier.cpp | 4 ++++ llvm/test/Verifier/DILocation-scope.ll | 24 ++++++++++++++++++++++++ 2 files changed, 28 insertions(+) create mode 100644 llvm/test/Verifier/DILocation-scope.ll diff --git a/llvm/lib/IR/Verifier.cpp b/llvm/lib/IR/Verifier.cpp index 5bc731c32664..2dcb9c4adf8d 100644 --- a/llvm/lib/IR/Verifier.cpp +++ b/llvm/lib/IR/Verifier.cpp @@ -2264,6 +2264,10 @@ void Verifier::visitFunction(const Function &F) { if (!Seen.insert(DL).second) continue; + Metadata *Parent = DL->getRawScope(); + AssertDI(Parent && isa(Parent), + "DILocation's scope must be a DILocalScope", N, &F, &I, DL, + Parent); DILocalScope *Scope = DL->getInlinedAtScope(); if (Scope && !Seen.insert(Scope).second) continue; diff --git a/llvm/test/Verifier/DILocation-scope.ll b/llvm/test/Verifier/DILocation-scope.ll new file mode 100644 index 000000000000..5cac5aead82d --- /dev/null +++ b/llvm/test/Verifier/DILocation-scope.ll @@ -0,0 +1,24 @@ +; RUN: llvm-as -disable-output %s -o - 2>&1 | FileCheck %s +source_filename = "t.c" +target datalayout = "e-m:o-i64:64-f80:128-n8:16:32:64-S128" +target triple = "x86_64-apple-macosx10.13.0" + +define void @f() !dbg !4 { +entry: +; CHECK: scope must be a DILocalScope +; CHECK: DILocation + ret void, !dbg !6 +} + +; CHECK: warning: ignoring invalid debug info + +!llvm.dbg.cu = !{!0} +!llvm.module.flags = !{!2, !3} + +!0 = distinct !DICompileUnit(language: DW_LANG_C99, file: !1, emissionKind: FullDebug) +!1 = !DIFile(filename: "t.c", directory: "/tmp") +!2 = !{i32 2, !"Dwarf Version", i32 4} +!3 = !{i32 2, !"Debug Info Version", i32 3} +!4 = distinct !DISubprogram(name: "f", scope: !1, file: !1, line: 2, type: !5, isDefinition: true, scopeLine: 2, flags: DIFlagPrototyped, unit: !0) +!5 = !DISubroutineType(types: !{}) +!6 = !DILocation(line: 2, scope: !1)