Discussion:
Use _LDBL_EQ_DBL in nexttowardf.c
Tom de Vries
2018-05-07 09:39:10 UTC
Permalink
Hi,

quoting from here ( https://sourceware.org/ml/newlib/2014/msg00688.html ):
...
Drop the FORCE_EVAL macro, and wrap nexttowardf in _LDBL_EQ_DBL as
per > Craig's suggestion.
...

Actually, the patch that Jon submitted there is the same as before, and
is missing those changes.

The commit does contain those changes, except that it uses the
non-existing macro _LDBL_EQ_DOUBLE instead of _LDBL_EQ_DBL.

This patch fixes that.

OK?

[ Found this by running into a link error "unresolved symbol
nexttowardf" for gcc testcase gcc/testsuite/gcc.dg/nextafter-2.c using
the standalone nvptx toolchain. ]

Thanks,
- Tom
Jeff Johnston
2018-05-07 16:23:46 UTC
Permalink
Patch applied.

-- Jeff J.
Post by Tom de Vries
Hi,
...
Drop the FORCE_EVAL macro, and wrap nexttowardf in _LDBL_EQ_DBL as per >
Craig's suggestion.
...
Actually, the patch that Jon submitted there is the same as before, and is
missing those changes.
The commit does contain those changes, except that it uses the
non-existing macro _LDBL_EQ_DOUBLE instead of _LDBL_EQ_DBL.
This patch fixes that.
OK?
[ Found this by running into a link error "unresolved symbol nexttowardf"
for gcc testcase gcc/testsuite/gcc.dg/nextafter-2.c using the standalone
nvptx toolchain. ]
Thanks,
- Tom
Loading...