[PATCH] D25417: [libcxxabi] Fix alignment of allocated exceptions in 32 bit builds
Saleem Abdulrasool via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Dec 5 10:16:28 PST 2016
compnerd accepted this revision.
compnerd added a comment.
This revision is now accepted and ready to land.
I thought we had gotten this merged already. Oops. LGTM with the minor tweak for windows.
================
Comment at: src/fallback_malloc.cpp:206
+ return fallback_malloc(size);
}
----------------
Given that libc++abi has some Windows support and usage (e.g. MinGW), we should change this to:
void *dest;
#if defined(_WIN32)
if ((dest = _aligned_malloc(size, alignof(__aligned_type)) == nullptr)
return fallback_malloc(size);
return dest;
#else
if (::posix_memalign(&dest, alignof(__aligned_type), size) == 0)
return dest;
return fallback_malloc(size);
#endif
https://reviews.llvm.org/D25417
More information about the cfe-commits
mailing list