On Wed, Dec 21, 2016 at 4:49 AM, Mark Kettenis <mark.kette...@xs4all.nl> wrote: >> Date: Wed, 21 Dec 2016 13:28:26 +0100 >> From: Christian Weisgerber <na...@mips.inka.de> >> >> libm uses copysign() and copysignf() internally, but fails to declare >> the amd64 assembly versions that way. When built with clang, this >> results in undefined references to _libm_copysign etc. >> >> Presumably gcc replaces those calls to copysign with a builtin, but >> clang doesn't. > > Looks correct to me. Hopefully guenther@ can confirm?
Looks right. Might as well make the same change in libm/arch/i387/ so that they stay in sync for the symbols that they provide. ok guenther@ (Hmm, copysign(3) doesn't mention that it's in C99)