summary | shortlog | log | commit | commitdiff | tree
raw | patch | inline | side by side (parent: 4f7e2c3)
raw | patch | inline | side by side (parent: 4f7e2c3)
author | Manman Ren <manman.ren@gmail.com> | |
Wed, 11 Sep 2013 19:40:28 +0000 (19:40 +0000) | ||
committer | Manman Ren <manman.ren@gmail.com> | |
Wed, 11 Sep 2013 19:40:28 +0000 (19:40 +0000) |
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@190544 91177308-0d34-0410-b5e6-96231b3b80d8
lib/CodeGen/AsmPrinter/DwarfDebug.cpp | patch | blob | history |
index 6f6a7de5daa37d440a323d4d4d4daeb8a4dab169..61c02b8407981bf79c0d49e1299f254578d50b3e 100644 (file)
return SPDie;
}
+/// Check whether we should create a DIE for the given Scope, return true
+/// if we don't create a DIE (the corresponding DIE is null).
bool DwarfDebug::isLexicalScopeDIENull(LexicalScope *Scope) {
if (Scope->isAbstractScope())
return false;
+ // We don't create a DIE if there is no Range.
const SmallVectorImpl<InsnRange> &Ranges = Scope->getRanges();
if (Ranges.empty())
return true;
if (Ranges.size() > 1)
return false;
+ // We don't create a DIE if we have a single Range and the end label
+ // is null.
SmallVectorImpl<InsnRange>::const_iterator RI = Ranges.begin();
MCSymbol *End = getLabelAfterInsn(RI->second);
return !End;
DIE *ObjectPointer = NULL;
bool ChildrenCreated = false;
+ // We try to create the scope DIE first, then the children DIEs. This will
+ // avoid creating un-used children then removing them later when we find out
+ // the scope DIE is null.
DIE *ScopeDIE = NULL;
if (Scope->getInlinedAt())
ScopeDIE = constructInlinedScopeDIE(TheCU, Scope);
ScopeDIE = updateSubprogramScopeDIE(TheCU, DS);
}
else {
+ // Early exit when we know the scope DIE is going to be null.
if (isLexicalScopeDIENull(Scope))
return NULL;
- // We create children only when we know the scope DIE is not going to be
- // null.
+
+ // We create children here when we know the scope DIE is not going to be
+ // null and the children will be added to the scope DIE.
ObjectPointer = createScopeChildrenDIE(TheCU, Scope, Children);
ChildrenCreated = true;
+
// There is no need to emit empty lexical block DIE.
std::pair<ImportedEntityMap::const_iterator,
ImportedEntityMap::const_iterator> Range = std::equal_range(
return NULL;
}
if (!ChildrenCreated)
+ // We create children when the scope DIE is not null.
ObjectPointer = createScopeChildrenDIE(TheCU, Scope, Children);
// Add children