[PATCH] [ms-cxxabi] Correctly compute the size of member pointers

Reid Kleckner rnk at google.com
Tue Mar 26 10:01:23 PDT 2013


Is this patch good to go, or did I add too much to it?  In particular, I
had to make changes to the clang attribute tablegen, so I'd rather not
commit that without review.


On Fri, Mar 22, 2013 at 4:50 PM, Reid Kleckner <rnk at google.com> wrote:

>     - Implement John's comments.
>
> Hi rjmccall,
>
> http://llvm-reviews.chandlerc.com/D568
>
> CHANGE SINCE LAST DIFF
>   http://llvm-reviews.chandlerc.com/D568?vs=1374&id=1376#toc
>
> Files:
>   include/clang/AST/Attr.h
>   include/clang/Basic/Attr.td
>   include/clang/Basic/AttrKinds.h
>   lib/AST/ASTContext.cpp
>   lib/AST/AttrImpl.cpp
>   lib/AST/CXXABI.h
>   lib/AST/ItaniumCXXABI.cpp
>   lib/AST/MicrosoftCXXABI.cpp
>   lib/Sema/SemaType.cpp
>   test/SemaCXX/member-pointer-ms.cpp
>   utils/TableGen/ClangAttrEmitter.cpp
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130326/5be67ea4/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D568.3.patch
Type: application/octet-stream
Size: 17666 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130326/5be67ea4/attachment.obj>


More information about the cfe-commits mailing list