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

Sebastian Huber sebastian.huber@embedded-brains.de
Thu Aug 9 04:56:00 GMT 2018


On 08/08/18 16:33, Jonathan Wakely wrote:
> 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. 

Should I check in my patch in addition to your patch?

-- 
Sebastian Huber, embedded brains GmbH

Address : Dornierstr. 4, D-82178 Puchheim, Germany
Phone   : +49 89 189 47 41-16
Fax     : +49 89 189 47 41-09
E-Mail  : sebastian.huber@embedded-brains.de
PGP     : Public key available on request.

Diese Nachricht ist keine geschäftliche Mitteilung im Sinne des EHUG.



More information about the Libstdc++ mailing list