[PATCH] D40277: [MS] Increase default new alignment for win64 and test it

2017-11-20 Thread Reid Kleckner via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL318723: [MS] Increase default new alignment for win64 and test it (authored by rnk). Changed prior to commit: https://reviews.llvm.org/D40277?vs=123692=123698#toc Repository: rL LLVM

[PATCH] D40277: [MS] Increase default new alignment for win64 and test it

2017-11-20 Thread Richard Smith - zygoloid via Phabricator via cfe-commits
rsmith accepted this revision. rsmith added a comment. This revision is now accepted and ready to land. Looks fine to me, but I think the alignment numbers in your summary are off by a factor of two. https://reviews.llvm.org/D40277 ___ cfe-commits

[PATCH] D40277: [MS] Increase default new alignment for win64 and test it

2017-11-20 Thread Reid Kleckner via Phabricator via cfe-commits
rnk created this revision. This raises __STDCPP_DEFAULT_NEW_ALIGNMENT__ from 16 to 32 on Win64. This matches platforms that follow the usual `2 * sizeof(void*)` alignment requirement for malloc. We might want to consider making that the default rather than relying on long double alignment. Fixes