[PATCH] [ms-cxxabi] Implement vbtable name mangling

[Reid Kleckner]

On Thu, Jun 6, 2013 at 9:00 PM, John McCall <rjmccall at apple.com> wrote:

> +typedef llvm::ArrayRef<const CXXRecordDecl *> ClassArrayRef;
> +
>
> Don't do this.  It's really not saving you anything, and it makes the API
> randomly harder to understand.
>

I blame Timur.  :)


> --- lib/CodeGen/CGVTables.cpp
> +++ lib/CodeGen/CGVTables.cpp
>
> Is this benefitting from being in this file?  I actually want to eliminate
> CGVTables in the long run and replace it with minimal-surface-area
> builder classes specific to, and invoked by, the ABI classes.  If you
> can do something more along those lines, I'd be happier.  Think
> smaller, more single-purpose implementation files, with CGM-level
> state kept in the ABI class.
>

So, something along the lines of CodeGen/MicrosoftVBTables.cpp?


> I'm happy that you seem to have figured out the disambiguation logic.
> That part all looks fine, although I haven't really reviewed your
> algorithm.
>
> John.
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130610/46b2f234/attachment.html>