<div dir="ltr">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 every type)?</div><br><div class="gmail_quote"><div dir="ltr">On Fri, Apr 8, 2016 at 3:24 PM Greg Clayton <<a href="mailto:clayborg@gmail.com">clayborg@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">clayborg added a comment.<br>
<br>
In <a href="http://reviews.llvm.org/D18848#395933" rel="noreferrer" target="_blank">http://reviews.llvm.org/D18848#395933</a>, @zturner wrote:<br>
<br>
> I mean for example if someone calls `FindTypes(..., "char", ...);`<br>
><br>
> Should a `TypeSP` go into the map?<br>
<br>
<br>
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).<br>
<br>
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.<br>
<br>
<br>
<a href="http://reviews.llvm.org/D18848" rel="noreferrer" target="_blank">http://reviews.llvm.org/D18848</a><br>
<br>
<br>
<br>
</blockquote></div>