Re: [PATCH 1/7] newlib: libm: merge machine/ trampoline up a level

[Mike Frysinger <vapier@gentoo.org>]

[-- Attachment #1: Type: text/plain, Size: 973 bytes --]

On 27 Jan 2022 15:28, Jon Turney wrote:
> I hadn't noticed before, but there also seems to be something in this 
> series which seems to break building for x86.

i haven't been able to get an i686 or x86_64 build of newlib working at all.
like, not even before i landed any commits.  so it's been hard for me to test
those targets specifically.  cygwin in particular doesn't work out of the box
as building gcc blows up :(.

i sent you a small patch series which should fix this though.

> (see https://github.com/cygwin/cygwin/actions/runs/1754176435)
> 
> >   CC       lib_a-f_exp.o
> > In file included from ../../../../../../newlib/libm/machine/i386/f_exp.c:29:
> > ../../../../../../newlib/libm/machine/i386/f_math.h:5:10: fatal error: fdlibm.h: No such file or directory
> >     5 | #include "fdlibm.h"
> >       |          ^~~~~~~~~~

you prob want to run --disable-silent-rules in your CI system so the full
compile commands are logged.
-mike

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]