[PATCH] [ms-cxxabi] Correctly compute the size of member pointers
John McCall
rjmccall at apple.com
Thu Mar 28 10:49:55 PDT 2013
On Mar 28, 2013, at 9:27 AM, Reid Kleckner <rnk at google.com> wrote:
> This patch looks pretty close to done to me. I started doing more codegen for these, and I'd like some way to share the code between lib/AST/MicrosoftCXXABI.cpp and lib/CodeGen/MicrosoftCXXABI.cpp. In particular, I want to share the inheritance model computation stuff. Any ideas on the best way to do that?
>
> I could add CXXRecordDecl/MemberPointerType::getMSInheritanceModel(), if that's an OK place to put it.
Putting it on CXXRecordDecl makes sense. You should make it return a proper enum, though, not an attribute kind.
John.
More information about the cfe-commits
mailing list