EricWF added a comment.

In http://reviews.llvm.org/D12512#237175, @joerg wrote:
> Please don't commit this as is. Many platforms have posix_memalign or 
> equivalent, which makes this both simpler and potentially without wasting 
> memory. Compare e.g. http://reviews.llvm.org/D12001.


Will do. Any advice on detecting posix_memalign? I don't see anything in the 
patch you pointed me to.

PS. Thanks for actually rejecting the patch. It makes the phab workflow easier 
to use.


http://reviews.llvm.org/D12512



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to