From 461a646a035a8c5cf717fcd006db0d637e342e70 Mon Sep 17 00:00:00 2001 From: Devang Patel Date: Wed, 19 May 2010 21:58:28 +0000 Subject: [PATCH] Revert r104165. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@104172 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/CodeGen/AsmPrinter/DwarfDebug.cpp | 16 +++++++++++----- lib/CodeGen/AsmPrinter/DwarfDebug.h | 2 ++ 2 files changed, 13 insertions(+), 5 deletions(-) diff --git a/lib/CodeGen/AsmPrinter/DwarfDebug.cpp b/lib/CodeGen/AsmPrinter/DwarfDebug.cpp index 4287c115bdd..ce13945583e 100644 --- a/lib/CodeGen/AsmPrinter/DwarfDebug.cpp +++ b/lib/CodeGen/AsmPrinter/DwarfDebug.cpp @@ -2230,8 +2230,11 @@ void DwarfDebug::beginScope(const MachineInstr *MI) { Label = recordSourceLine(DL.getLine(), DL.getCol(), 0); PrevInstLoc = DL; PrevLabel = Label; - LabelsBeforeInsn[MI] = Label; } + + // If this instruction begins a scope then note down corresponding label. + if (InsnsBeginScopeSet.count(MI) != 0) + LabelsBeforeInsn[MI] = Label; } return; @@ -2255,7 +2258,6 @@ void DwarfDebug::beginScope(const MachineInstr *MI) { Label = recordSourceLine(DL.getLine(), DL.getCol(), Scope); PrevInstLoc = DL; PrevLabel = Label; - LabelsBeforeInsn[MI] = Label; } DI->second->setDbgValueLabel(Label); } @@ -2271,9 +2273,11 @@ void DwarfDebug::beginScope(const MachineInstr *MI) { Label = recordSourceLine(DL.getLine(), DL.getCol(), Scope); PrevInstLoc = DL; PrevLabel = Label; - LabelsBeforeInsn[MI] = Label; } + // If this instruction begins a scope then note down corresponding label. + if (InsnsBeginScopeSet.count(MI) != 0) + LabelsBeforeInsn[MI] = Label; } /// endScope - Process end of a scope. @@ -2531,8 +2535,9 @@ void DwarfDebug::identifyScopeMarkers() { continue; for (SmallVector::const_iterator RI = Ranges.begin(), RE = Ranges.end(); RI != RE; ++RI) { - assert(RI->first && "DbgRange does not have first instruction!"); - assert(RI->second && "DbgRange does not have second instruction!"); + assert(RI->first && "DbgRange does not have first instruction!"); + assert(RI->second && "DbgRange does not have second instruction!"); + InsnsBeginScopeSet.insert(RI->first); InsnsEndScopeSet.insert(RI->second); } } @@ -2626,6 +2631,7 @@ void DwarfDebug::endFunction(const MachineFunction *MF) { // Clear debug info CurrentFnDbgScope = NULL; DeleteContainerSeconds(DbgScopeMap); + InsnsBeginScopeSet.clear(); InsnsEndScopeSet.clear(); DbgValueStartMap.clear(); ConcreteScopes.clear(); diff --git a/lib/CodeGen/AsmPrinter/DwarfDebug.h b/lib/CodeGen/AsmPrinter/DwarfDebug.h index 2c4c9b66001..48a52623c39 100644 --- a/lib/CodeGen/AsmPrinter/DwarfDebug.h +++ b/lib/CodeGen/AsmPrinter/DwarfDebug.h @@ -179,6 +179,8 @@ class DwarfDebug { /// corresponds to the MDNode mapped with the subprogram DIE. DenseMap ContainingTypeMap; + typedef SmallVector ScopeVector; + SmallPtrSet InsnsBeginScopeSet; SmallPtrSet InsnsEndScopeSet; /// InlineInfo - Keep track of inlined functions and their location. This