mirror of
https://github.com/c64scene-ar/llvm-6502.git
synced 2025-06-28 06:24:57 +00:00
Temporarily revert "Don't try to construct debug LexicalScopes hierarchy for functions that do not have top level debug information." as it appears to be breaking some LTO constructs.
This reverts commit r212203. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@212298 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
@ -730,8 +730,6 @@ void DwarfDebug::beginModule() {
|
||||
|
||||
const Module *M = MMI->getModule();
|
||||
|
||||
FunctionDIs = makeSubprogramMap(*M);
|
||||
|
||||
// If module has named metadata anchors then use them, otherwise scan the
|
||||
// module using debug info finder to collect debug info.
|
||||
NamedMDNode *CU_Nodes = M->getNamedMetadata("llvm.dbg.cu");
|
||||
@ -1417,10 +1415,6 @@ void DwarfDebug::beginFunction(const MachineFunction *MF) {
|
||||
if (!MMI->hasDebugInfo())
|
||||
return;
|
||||
|
||||
auto DI = FunctionDIs.find(MF->getFunction());
|
||||
if (DI == FunctionDIs.end())
|
||||
return;
|
||||
|
||||
// Grab the lexical scopes for the function, if we don't have any of those
|
||||
// then we're not going to be able to do anything.
|
||||
LScopes.initialize(*MF);
|
||||
@ -1436,7 +1430,6 @@ void DwarfDebug::beginFunction(const MachineFunction *MF) {
|
||||
// belongs to so that we add to the correct per-cu line table in the
|
||||
// non-asm case.
|
||||
LexicalScope *FnScope = LScopes.getCurrentFunctionScope();
|
||||
assert(FnScope->getScopeNode() == DI->second);
|
||||
DwarfCompileUnit *TheCU = SPMap.lookup(FnScope->getScopeNode());
|
||||
assert(TheCU && "Unable to find compile unit!");
|
||||
if (Asm->OutStreamer.hasRawTextSupport())
|
||||
@ -1534,8 +1527,7 @@ void DwarfDebug::endFunction(const MachineFunction *MF) {
|
||||
assert(CurFn == MF);
|
||||
assert(CurFn != nullptr);
|
||||
|
||||
if (!MMI->hasDebugInfo() || LScopes.empty() ||
|
||||
!FunctionDIs.count(MF->getFunction())) {
|
||||
if (!MMI->hasDebugInfo() || LScopes.empty()) {
|
||||
// If we don't have a lexical scope for this function then there will
|
||||
// be a hole in the range information. Keep note of this by setting the
|
||||
// previously used section to nullptr.
|
||||
|
Reference in New Issue
Block a user