[Lldb-commits] [PATCH] D110570: [lldb] Refactor variable parsing in DWARF symbol file
Jaroslav Sevcik via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Mon Sep 27 10:57:10 PDT 2021
jarin added a comment.
Hi, could you possibly take a look at this change?
The main motivation for this patch is to move the setup of the variable list for parsing children to the DIE node that corresponds to the block containing the children. This will be particularly important for DW_TAG_inlined_subroutine because after a recent clang change (https://reviews.llvm.org/D95617), we can have inlined subroutines which contain no parameters (and thus no children) in the concrete DIE block, but still have some parameters in the abstract function (referenced by the inlined subroutine's DW_AT_abstract_origin attribute). If we started building the variable list when parsing the first variable in a function, we would never create the variable lists for the abstract parameters if they were all omitted.
Note that this change is just a refactoring for the actual change that will introduce parsing of the omitted abstract parameters. The actual change will be uploaded shortly.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D110570/new/
https://reviews.llvm.org/D110570
More information about the lldb-commits
mailing list