================ Comment at: include/clang/AST/RecordLayout.h:87 @@ +86,3 @@ + /// the __declspec(align()) trumps #pramga pack and must always be obeyed. + CharUnits RequiredAlignment; + ---------------- What about record subobjects in plain C? Don't they also have this property? Seems like it needs to live on ASTRecordLayout.
================ Comment at: lib/AST/RecordLayoutBuilder.cpp:2156 @@ +2155,3 @@ + if (!MaxFieldAlignment.isZero()) + BaseAlignment = std::min(BaseAlignment, + std::max(MaxFieldAlignment, ---------------- Remind me why this is a min? http://llvm-reviews.chandlerc.com/D2283 _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
