Re: Wrong unconditional dependency on nanl

Corinna Vinschen <[email protected]>
Newsgroups gmane.comp.lib.newlib
Message-ID <[email protected]>
On Oct 10 11:22, Craig Howland wrote:
> On 10/10/2018 10:52 AM, Corinna Vinschen wrote:
> > On Oct 10 15:48, Christophe Lyon wrote:
> > > On Wed, 10 Oct 2018 at 11:41, Corinna Vinschen <[email protected]> wrote:
> > > > On Oct  8 15:06, Christophe Lyon wrote:
> > > > 
> > > It does help on aarch64-elf, but a change to rdimon.specs is required
> > > because libc now depends on libm: this can be seen when compiling a
> > > C++ hello.cpp.
> > Hmm, that's an interesting point.  What I don't understand is why this
> > dependency wasn't already visible before.  strtod.c and wcstod.c call
> > nan() and nanf(), vfprintf.c and vfwprintf.c call nanf().  Why is a
> > call to nanl() different?
> The nanl() call was only added perhaps a few weeks ago.  At that time we
> missed the thought that it comes from libm.  We'd need to add it to
> newlib/Makefile to be copied like s_nan and sf_nan presently are.

Yeah, just found this myself.  nan/nanf are pulled in via newlib's
top-level Makefile.am:

  MATHOBJS_IN_LIBC = ...

Pulling in nanl is a problem because it's build depends on
HAVE_LONG_DOUBLE.

Maybe we should really just use __builtin_nanl in strtorx.c...?


Corinna

-- 
Corinna Vinschen
Cygwin Maintainer
Red Hat
signature.asc (application/pgp-signature, 833 B)
-----BEGIN PGP SIGNATURE-----

iQIzBAEBCAAdFiEEoVYPmneWZnwT6kwF9TYGna5ET6AFAlu+G2sACgkQ9TYGna5E
T6DXCg/+MNOs94rgBpxwjy/Z2pukpRlT4DckXvYEGGkY2FGhiIto6Ifyqsl0Wxuc
gJVTZWAM3cmODVv5MLoEX8ndwNae8Y0AXUt08lXVQ7BsO1ifQSVPsXe1PEsajV5P
GIp5BStwTJMXK2lyWRMq862RYj4ee7rhXNPPs4rmFnJIyHNNxJJIhDfw/a78aGve
N/O++fbWRRP3T+EhyR8ppFSKvzkpEB5FCxDAqsXDl9XJKDYq+TvGWNP8s8oXfGU6
fmt5UyY2E1YfeFgraa4WECrF1ka2D32J2u+iXtAQBLpwvqzPyYmdjSKV/jqlVyv1
1nWUlYb3GiayEVAAF/MPEWKeiBNjiFUuOu8qGNBa5TkR191JGHbMXjiO0UowQ/ko
Vf/WnTJwgMwU3Jn6/WC6uwZnU+3E+U6NzoHVV7d1KrQwwyfJhNv6ZH0BZ1ME4BT0
crrbZBqTsNpal0SdjtD+AohQUYaNSYOkdZM/hCqx9FPceUWqLRLMAkap+JTBxgtn
cFc5j0dk0JaXHv/ZdB9jyDG552sft42ru0Pt+P4dt8MIu+pECP8aTbrfsyKDByfG
8LjPLouVM7jvXDx+Y6QpIlSS/AUrQv0C5PuRBH5JGbNra5GpRsHxmLoZ2v+QDP6J
nRrEoVnSiFuK+fNZvjgijQlkzlLnUT0alH7nFaXNpSxBcglGynA=
=VU0P
-----END PGP SIGNATURE-----
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.