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

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

-----BEGIN PGP SIGNATURE-----

iQIzBAABCAAdFiEEuQK1JxMl+JKsJRrUQWM7n+g39YEFAmHzOtMACgkQQWM7n+g3
9YHiKxAAv4i4vZDMEQww5AH8Umb6OxFde3X+1cVmxhuF8w0uf2SuUEzqTDWHEMix
B9Y9cMwn4fpiQXggVmyGg291qWBuEsCiyfBpBIeWK/o7YcyQOe6nVbzp5Is4z1i+
Bdyh1qVsC59FU0tweBamTe2Nz/s8f3gu9omt4SPfZxj0LoVHroeOFn2yOJScuUaL
Q3dO0KeJoteOdw5pubVCeL90uKi/ewOkcx9AYqf2XvTDBV/LcNIdbJS10bqs6JeU
ZaVTotKenAv6SjNeWU//AcBj+f6iMoVkQF/l/tpefzOuOFT+BYIfvDQG1X5SsLIV
M/+TEjvLXBPp1TNeb3qRlFvtnxCjz+8loiViheT4jVWxpuVLNERw27SKzJUKppes
lN4JsSDrKFWTboea7RGPiet5lrAibTSKDkOXqaK0QlCJX6UQDz9nljgEL7U3kKp3
Q+fPnC8w8FUhS+YGaxOXYw6HxZ+LK9svCRERwDDD4iGYRUAdqABR1D23oRGxWL8m
50cL3HqKhvbwxjo5tiYQrtTr2bk9xCH6Ya6pto2ueSYBmDPCFoC4nSewqusgNZrP
hOmPqw/qgcToc9k/Me7Qhw5JRQ5GVDbXXmOD0xOZL0GTA7lz0jiGm3WYw4pnDKIO
hMOsdqM3GIxJ2yLrzS6bXdCr7ugCsnSMOE4QPk+9rrhOJAYFDv0=
=GGSi
-----END PGP SIGNATURE-----