On 18 August 2013 15:27, Iain Buclaw <ibuc...@ubuntu.com> wrote: > On 18 August 2013 09:54, Johannes Pfau <nos...@example.com> wrote: >> Am Fri, 16 Aug 2013 18:38:08 +0200 >> schrieb "Iain Buclaw" <ibuc...@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';
