<p dir="ltr">Android has posix_memalign too.</p>
<div class="gmail_quote">On Sep 5, 2015 12:00, "Joerg Sonnenberger" <<a href="mailto:joerg@britannica.bec.de">joerg@britannica.bec.de</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Fri, Sep 04, 2015 at 09:28:39PM +0000, Eric Fiselier via cfe-commits wrote:<br>
> EricWF added a comment.<br>
><br>
> In <a href="http://reviews.llvm.org/D12512#237175" rel="noreferrer" target="_blank">http://reviews.llvm.org/D12512#237175</a>, @joerg wrote:<br>
><br>
> > 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. <a href="http://reviews.llvm.org/D12001" rel="noreferrer" target="_blank">http://reviews.llvm.org/D12001</a>.<br>
><br>
><br>
> Will do. Any advice on detecting posix_memalign? I don't see anything in the patch you pointed me to.<br>
<br>
Provide a feature variable for it and set it by default. For Unix-like<br>
systems, I think only Android wants to disable it by default. Not sure<br>
about Windows and OSX.<br>
<br>
Joerg<br>
</blockquote></div>