Android has posix_memalign too. On Sep 5, 2015 12:00, "Joerg Sonnenberger" <jo...@britannica.bec.de> wrote:
> On Fri, Sep 04, 2015 at 09:28:39PM +0000, Eric Fiselier via cfe-commits > wrote: > > 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. > > Provide a feature variable for it and set it by default. For Unix-like > systems, I think only Android wants to disable it by default. Not sure > about Windows and OSX. > > Joerg >
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
