[Lldb-commits] [lldb] [LLDB] Ignore actual-needed artificial members in DWARFASTParserClang::ParseSingleMember (PR #70779)

[Michael Buch via lldb-commits]

================
@@ -771,6 +771,10 @@ TypeSystemClang *TypeSystemClang::GetASTContext(clang::ASTContext *ast) {
   return clang_ast;
 }
 
+bool TypeSystemClang::ShouldIgnoreArtificialField(llvm::StringRef Name) {
+  return Name.starts_with("_vptr$");
----------------
Michael137 wrote:

`TypeSystemClang` should be agnostic to the debug-info format. The various debug-info->AST parsers (e.g., `DWARFASTParserClang`/`PDBASTParser`/`npdb::PdbAstBuilder`, the latter two being the MSVC debug-info parsers) use `TypeSystemClang` after they've parsed debug-info to construct clang AST nodes. But I'm not sure how the PDB parsers handle the vptr case and artificial variables. From a brief glance it doesn't look like it cares about artificial member variables at all.

https://github.com/llvm/llvm-project/pull/70779