ARM testers, make note.

[Iain Buclaw]

On 18 August 2013 15:27, Iain Buclaw <ibuclaw at ubuntu.com> wrote:
> On 18 August 2013 09:54, Johannes Pfau <nospam at example.com> wrote:
>> Am Fri, 16 Aug 2013 18:38:08 +0200
>> schrieb "Iain Buclaw" <ibuclaw at ubuntu.com>:
>>
>>> On Friday, 16 August 2013 at 16:33:17 UTC, Iain Buclaw wrote:
>>> > I've pushed in pure implementations of functions in std.math,
>>> > removing the workaround in core.stdc.math.
>>> >
>>> > This passes for x64/x32. Can any ARM testers please report any
>>> > problems causecd by this update?
>>> >
>>> > Thanks
>>> > Iain.
>>>
>>> Commit reference:
>>> https://github.com/D-Programming-GDC/GDC/commit/a4e0b2928b92fae3448ec541f8ec86ffea4c8395
>>>
>>
>> Compiling phobos fails with this message:
>>
>> ../../../../gcc-4.8.1/libphobos/src/std/math.d:3264: Error: static
>> assert  "Only 80-bit and 96-bit reals are supported by lrint()"
>>
>> real is 64 bit on most (all?) ARM machines.
>>
>>
>> I also started the compiler test suite some time ago but it will take
>> some more time till it's finished.
>
>
> OK, fixed that:
> https://github.com/D-Programming-GDC/GDC/commit/196fdf37b3b3e7d25b09a1cac1f30e313274e6e5
>

Made the tiniest amendment:
https://github.com/D-Programming-GDC/GDC/commit/05b6e48d5d7736c7bcaeff9d71c807bcd0132dca


Should be all good after some preliminary testing vs __builtin_lrintl() results.

-- 
Iain Buclaw

*(p < e ? p++ : p) = (c & 0x0f) + '0';