[Lldb-commits] [PATCH] D18848: Add PDBASTParser and parse type information from PDB
Zachary Turner via lldb-commits
lldb-commits at lists.llvm.org
Mon Apr 11 15:08:55 PDT 2016
Thanks. One more question. Does SymbolFile::FindTypes need to be able to
handle the case of a null type name (which I guess would indicate to find
On Fri, Apr 8, 2016 at 3:24 PM Greg Clayton <clayborg at gmail.com> wrote:
> clayborg added a comment.
> In http://reviews.llvm.org/D18848#395933, @zturner wrote:
> > I mean for example if someone calls `FindTypes(..., "char", ...);`
> > Should a `TypeSP` go into the map?
> No you don't have to make one up if you don't have one. At least this is
> what the DWARF parser does. The main problem is the definition for "char"
> can be signed or unsigned depending on how your translation unit was
> compiled. Probably some other char types can vary as well. No expressions
> will ask for built in types since the expression parser already knows about
> them. And it is useful to try and lookup "char" in a SymbolFile to see what
> the notion that that symbol file has for "char" (signed or unsigned).
> So if you actually have one, return it. Or if PDB always has some around
> and it has some unique identifiers for the builtin types you can easily
> create them from your clang::ASTContext with the "ast->IntTy" and the many
> other built int types. But if you don't have any, don't worry about
> creating them from nothing.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the lldb-commits