DILexicalBlockFile should be skipped and use its DILexicalBlock parent scope when considering "entities declared in lexical scope".
This patch should fix https://llvm.org/bugs/show_bug.cgi?id=26942
And will be applied to changes in r263424.
Paths
| Differential D18350
Fixing DILexicalBlockFile issue in r263424 ClosedPublic Authored by aaboud on Mar 22 2016, 6:03 AM.
Details
Summary DILexicalBlockFile should be skipped and use its DILexicalBlock parent scope when considering "entities declared in lexical scope". This patch should fix https://llvm.org/bugs/show_bug.cgi?id=26942
Diff Detail
Event Timelineaaboud updated this object. Comment Actions Can you explain the problem that you are running into — why do you need to skip over the lexical block files (my guess is because they are not "real" lexical blocks)? The changes seem straightforward and fine to me, but a little bit of context would be helpful.
Comment Actions Please see the answer below.
aprantl edited edge metadata. Comment ActionsPutting this comment into the commit message works for me. Thanks! This revision is now accepted and ready to land.Mar 22 2016, 2:31 PM Closed by commit rL264280: Recommitted r263424 "Supporting all entities declared in lexical scope in… (authored by aaboud). · Explain WhyMar 24 2016, 6:35 AM This revision was automatically updated to reflect the committed changes. aaboud marked 3 inline comments as done. rnk added inline comments.
Comment Actions Since we have a public (if inconvenient) reproduction, can one of you Thanks! -eric
Revision Contents
Diff 51549 llvm/trunk/lib/CodeGen/AsmPrinter/DwarfCompileUnit.h
llvm/trunk/lib/CodeGen/AsmPrinter/DwarfCompileUnit.cpp
llvm/trunk/lib/CodeGen/AsmPrinter/DwarfDebug.h
llvm/trunk/lib/CodeGen/AsmPrinter/DwarfDebug.cpp
llvm/trunk/lib/CodeGen/AsmPrinter/DwarfUnit.h
llvm/trunk/lib/CodeGen/AsmPrinter/DwarfUnit.cpp
llvm/trunk/test/DebugInfo/Generic/2010-04-06-NestedFnDbgInfo.ll
llvm/trunk/test/DebugInfo/Generic/2010-06-29-InlinedFnLocalVar.ll
llvm/trunk/test/DebugInfo/Generic/cross-cu-linkonce-distinct.ll
llvm/trunk/test/DebugInfo/Generic/namespace.ll
llvm/trunk/test/DebugInfo/Generic/nodebug.ll
llvm/trunk/test/DebugInfo/Mips/InlinedFnLocalVar.ll
llvm/trunk/test/DebugInfo/X86/InlinedFnLocalVar.ll
llvm/trunk/test/DebugInfo/X86/PR24008.ll
llvm/trunk/test/DebugInfo/X86/dbg-file-name.ll
llvm/trunk/test/DebugInfo/X86/debug-dead-local-var.ll
llvm/trunk/test/DebugInfo/X86/gnu-public-names.ll
llvm/trunk/test/DebugInfo/X86/lexical-block-file.ll
llvm/trunk/test/DebugInfo/X86/lexical-block-inline.ll
llvm/trunk/test/DebugInfo/X86/lexical-block.ll
llvm/trunk/test/DebugInfo/X86/subprogram-inline.ll
|
This assert fails during a build of Chrome for iOS:
https://build.chromium.org/p/chromium.fyi/builders/ClangToTiOS/builds/6190/steps/compile/logs/stdio