maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

Revert Add debug location verification for !llvm.loop attachments. 

This reverts r363725 (git commit 8ff822d61d)

llvm-svn: 363728
This commit is contained in:
Adrian Prantl 2019-06-18 19:54:17 +00:00
parent 3fcad775c0
commit acc93d62e0
2 changed files with 28 additions and 64 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

64
llvm/lib/IR/Verifier.cpp
View File

@ -2342,44 +2342,36 @@ void Verifier::visitFunction(const Function &F) {
// FIXME: Check this incrementally while visiting !dbg attachments. // FIXME: Check this incrementally while visiting !dbg attachments.
// FIXME: Only check when N is the canonical subprogram for F. // FIXME: Only check when N is the canonical subprogram for F.
SmallPtrSet<const MDNode *, 32> Seen; SmallPtrSet<const MDNode *, 32> Seen;
auto VisitDebugLoc = [&](const Instruction &I, const MDNode *Node) {
// Be careful about using DILocation here since we might be dealing with
// broken code (this is the Verifier after all).
const DILocation *DL = dyn_cast_or_null<DILocation>(Node);
if (!DL)
return;
if (!Seen.insert(DL).second)
return;
Metadata *Parent = DL->getRawScope();
AssertDI(Parent && isa<DILocalScope>(Parent),
"DILocation's scope must be a DILocalScope", N, &F, &I, DL,
Parent);
DILocalScope *Scope = DL->getInlinedAtScope();
if (Scope && !Seen.insert(Scope).second)
return;
DISubprogram *SP = Scope ? Scope->getSubprogram() : nullptr;
// Scope and SP could be the same MDNode and we don't want to skip
// validation in that case
if (SP && ((Scope != SP) && !Seen.insert(SP).second))
return;
// FIXME: Once N is canonical, check "SP == &N".
AssertDI(SP->describes(&F),
"!dbg attachment points at wrong subprogram for function", N, &F,
&I, DL, Scope, SP);
};
for (auto &BB : F) for (auto &BB : F)
for (auto &I : BB) { for (auto &I : BB) {
VisitDebugLoc(I, I.getDebugLoc().getAsMDNode()); // Be careful about using DILocation here since we might be dealing with
// The llvm.loop annotations also contain two DILocations. // broken code (this is the Verifier after all).
if (auto MD = I.getMetadata(LLVMContext::MD_loop)) DILocation *DL =
for (unsigned i = 1; i < MD->getNumOperands(); ++i) dyn_cast_or_null<DILocation>(I.getDebugLoc().getAsMDNode());
VisitDebugLoc(I, dyn_cast_or_null<MDNode>(MD->getOperand(i))); if (!DL)
if (BrokenDebugInfo) continue;
return; if (!Seen.insert(DL).second)
continue;
Metadata *Parent = DL->getRawScope();
AssertDI(Parent && isa<DILocalScope>(Parent),
"DILocation's scope must be a DILocalScope", N, &F, &I, DL,
Parent);
DILocalScope *Scope = DL->getInlinedAtScope();
if (Scope && !Seen.insert(Scope).second)
continue;
DISubprogram *SP = Scope ? Scope->getSubprogram() : nullptr;
// Scope and SP could be the same MDNode and we don't want to skip
// validation in that case
if (SP && ((Scope != SP) && !Seen.insert(SP).second))
continue;
// FIXME: Once N is canonical, check "SP == &N".
AssertDI(SP->describes(&F),
"!dbg attachment points at wrong subprogram for function", N, &F,
&I, DL, Scope, SP);
} }
} }

28
llvm/test/Verifier/llvm.loop.ll
View File

@ -1,28 +0,0 @@
; RUN: llvm-as -disable-output < %s -o /dev/null 2>&1 | FileCheck %s
define i32 @foo() !dbg !4 {
entry:
ret i32 0, !dbg !6
}
define i32 @bar() !dbg !5 {
entry:
; CHECK: !dbg attachment points at wrong subprogram for function
ret i32 0, !dbg !10, !llvm.loop !9
}
; CHECK: warning: ignoring invalid debug info
!llvm.dbg.cu = !{!0}
!llvm.module.flags = !{!7, !8}
!0 = distinct !DICompileUnit(language: DW_LANG_C99, file: !1, producer: "clang", isOptimized: false, runtimeVersion: 0, emissionKind: FullDebug, enums: !2, retainedTypes: !2, globals: !2, imports: !2)
!1 = !DIFile(filename: "dwarf-test.c", directory: "test")
!2 = !{}
!4 = distinct !DISubprogram(name: "foo", scope: !0, isDefinition: true, unit: !0)
!5 = distinct !DISubprogram(name: "bar", scope: !0, isDefinition: true, unit: !0)
!6 = !DILocation(line: 7, scope: !4)
!7 = !{i32 2, !"Dwarf Version", i32 3}
!8 = !{i32 1, !"Debug Info Version", i32 3}
!9 = !{!9, !10, !11}
!10 = !DILocation(line: 1, scope: !5)
!11 = !DILocation(line: 1, scope: !4)