[Lldb-commits] [PATCH] D142413: [lldb] Don't create Clang AST nodes in GetCPlusPlusQualifiedName

Tue Jan 24 05:33:18 PST 2023

Michael137 added a comment.

In D142413#4075198 <https://reviews.llvm.org/D142413#4075198>, @aeubanks wrote:

> there may be an alternate solution involving making some declarations into definitions (at all? earlier?), but I don't have that level of understanding of lldb

One idea that came to mind is that instead of creating a dummy template decl in `GetDIEClassTemplateParams`, you could perhaps construct a `clang::TemplateArgumentList` from the `TemplateArgument`s stored in `TemplateParameterInfos` and use the `clang::printTemplateArgumentList` to stringify the template arguments, passing it the printing-policy which is used in `TypeSystemClang` to print type-names in your current implementation.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D142413/new/

https://reviews.llvm.org/D142413