[PATCH] libstdc++-v3: Have aligned_alloc() on Newlib

Jonathan Wakely jwakely@redhat.com
Wed Aug 8 14:33:00 GMT 2018


On 08/08/18 16:22 +0200, Ulrich Weigand wrote:
>Jonathan Wakely wrote:
>
>> Aha, so newlib was using memalign previously:
>>
>> @@ -53,20 +54,24 @@ aligned_alloc (std::size_t al, std::size_t sz)
>>  #else
>>  extern "C" void *memalign(std::size_t boundary, std::size_t size);
>>  #endif
>> -#define aligned_alloc memalign
>
>Yes, exactly ... this commit introduced the regression.
>
>> OK, I've regressed the branches then - I'll fix that.

This should fix it. I'll finish testing and commit it.

Sebastian, your patch to define HAVE_ALIGNED_ALLOC is OK for
gcc-7-branch and gcc-8-branch, because changing newlib from using
memalign to aligned_alloc is safe.

Sorry for the mess.




More information about the Libstdc++ mailing list