[Lldb-commits] [PATCH] D130942: [LLDB][DWARF] Set MSInheritanceAttr for record decl when it's using Microsoft compatible ABI.
Reid Kleckner via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Wed Aug 3 10:30:47 PDT 2022
rnk added inline comments.
================
Comment at: lldb/source/Plugins/SymbolFile/DWARF/DWARFASTParserClang.cpp:1754
+ m_ast.getASTContext(),
+ clang::MSInheritanceAttr::Spelling::Keyword_unspecified_inheritance));
+ }
----------------
I'm concerned that this isn't really the right fix. Changing the inheritance model changes the size of the member pointer representation, so the consequences of getting the wrong one could affect expression evaluation results. Zequan suggested guessing the inheritance model from the class definition, but we really ought to write down the inheritance model in the DWARF when using the MS ABI. This probably needs its own DWARF attribute.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D130942/new/
https://reviews.llvm.org/D130942
More information about the lldb-commits
mailing list