The second argument to RoundUpToAlignment is an alignment, not a size...
================ Comment at: include/clang/AST/StmtOpenMP.h:84 @@ +83,3 @@ + reinterpret_cast<char *>(this) + + llvm::RoundUpToAlignment(sizeof(T), sizeof(Expr *))), + NumVars); ---------------- The second argument here should be llvm::alignOf<Expr*>(), shouldn't it? ================ Comment at: include/clang/AST/StmtOpenMP.h:95 @@ -92,1 +94,3 @@ + reinterpret_cast<char *>(this) + + llvm::RoundUpToAlignment(sizeof(T), sizeof(Expr *)))); } ---------------- Same here (and several more below). http://llvm-reviews.chandlerc.com/D2713 _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
