<div dir="ltr">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?<div>
<br></div><div style>I could add CXXRecordDecl/MemberPointerType::getMSInheritanceModel(), if that's an OK place to put it.</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Mar 27, 2013 at 6:38 PM, Reid Kleckner <span dir="ltr"><<a href="mailto:rnk@google.com" target="_blank">rnk@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">    - Split function member pointer case from data case.<br>
<div class="im"><br>
Hi rjmccall,<br>
<br>
<a href="http://llvm-reviews.chandlerc.com/D568" target="_blank">http://llvm-reviews.chandlerc.com/D568</a><br>
<br>
CHANGE SINCE LAST DIFF<br>
</div>  <a href="http://llvm-reviews.chandlerc.com/D568?vs=1406&id=1416#toc" target="_blank">http://llvm-reviews.chandlerc.com/D568?vs=1406&id=1416#toc</a><br>
<div class="HOEnZb"><div class="h5"><br>
Files:<br>
  lib/AST/ASTContext.cpp<br>
  lib/AST/CXXABI.h<br>
  lib/AST/ItaniumCXXABI.cpp<br>
  lib/AST/MicrosoftCXXABI.cpp<br>
  lib/Sema/SemaType.cpp<br>
  test/SemaCXX/member-pointer-ms.cpp<br>
</div></div></blockquote></div><br></div>