efriedma added inline comments.
================ Comment at: cfe/trunk/lib/AST/RecordLayoutBuilder.cpp:1758 + Context.getTargetInfo().getTriple().isWindowsGNUEnvironment()) && + "Non PowerOf2 size outside of GNU mode"); + if (TypeSize > FieldAlign && ---------------- mstorsjo wrote: > efriedma wrote: > > This assertion seems weird. `sizeof(long double)` is 12 for other targets, > > including x86-32 Linux. > Perhaps it'd make more sense to flip it around, `assert(isPowerOf2() || > !Triple.isWindowsMSVCEnvironment())`? Yes, that makes sense. Repository: rL LLVM https://reviews.llvm.org/D43734 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
