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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
