[PATCH] Microsoft C++ Record Layout

Warren Hunt whunt at google.com
Tue Oct 8 14:25:51 PDT 2013


  Sure, I can add it but I don't think it's necessary.  It works fine with
  the patch (cl.exe and clang produce the same thing).  I didn't test the
  behavior pre-patch, I assume this was just another instance of the size of
  the vfptr not being "correctly" rounded up to the alignment of the struct
  (8 in this case), which is thoroughly tested.

  -Warren

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



More information about the cfe-commits mailing list