Skip to content

Commit

Permalink
Fix implicit __isinf declarations in exp.
Browse files Browse the repository at this point in the history
My recent exp patch introduced warnings about implicit __isinf
declarations in exp because e_exp.c didn't include <math.h>.  This
patch fixes this.  Because <math.h> can't be included after
<math_private.h> (because of macro definitions of __nan*), it was
necessary to put an include in sysdeps/x86_64/fpu/multiarch/e_exp.c as
well.

Tested x86_64.

	* sysdeps/ieee754/dbl-64/e_exp.c: Include <math.h>.
	* sysdeps/x86_64/fpu/multiarch/e_exp.c
	[HAVE_FMA4_SUPPORT || HAVE_AVX_SUPPORT]: Likewise.
  • Loading branch information
Joseph Myers committed Mar 24, 2014
1 parent a42faf5 commit f342689
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 0 deletions.
6 changes: 6 additions & 0 deletions ChangeLog
Original file line number Diff line number Diff line change
@@ -1,3 +1,9 @@
2014-03-24 Joseph Myers <joseph@codesourcery.com>

* sysdeps/ieee754/dbl-64/e_exp.c: Include <math.h>.
* sysdeps/x86_64/fpu/multiarch/e_exp.c
[HAVE_FMA4_SUPPORT || HAVE_AVX_SUPPORT]: Likewise.

2014-03-24 Paul Pluzhnikov <ppluzhnikov@google.com>

[BZ #16634]
Expand Down
1 change: 1 addition & 0 deletions sysdeps/ieee754/dbl-64/e_exp.c
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,7 @@
/* */
/***************************************************************************/

#include <math.h>
#include "endian.h"
#include "uexp.h"
#include "mydefs.h"
Expand Down
1 change: 1 addition & 0 deletions sysdeps/x86_64/fpu/multiarch/e_exp.c
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
#if defined HAVE_FMA4_SUPPORT || defined HAVE_AVX_SUPPORT
# include <init-arch.h>
# include <math.h>
# include <math_private.h>

extern double __ieee754_exp_sse2 (double);
Expand Down

0 comments on commit f342689

Please sign in to comment.