[PATCH] [ms-cxxabi] Raise aggregate memptr alignment to 8 for compatibility

Reid Kleckner rnk at google.com
Fri Jan 31 13:15:56 PST 2014


Hi majnemer,

With this change, we give different results for __alignof than MSVC, but
our record layout is compatible.

Some data member pointers also now have a size that is not a multiple of
their alignment.

Fixes PR18618.

http://llvm-reviews.chandlerc.com/D2669

Files:
  lib/AST/MicrosoftCXXABI.cpp
  test/CodeGenCXX/microsoft-abi-member-pointers.cpp
  test/Layout/ms-x86-member-pointers.cpp
  test/SemaCXX/member-pointer-ms.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D2669.1.patch
Type: text/x-patch
Size: 15108 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140131/9d9c525d/attachment.bin>


More information about the cfe-commits mailing list