From aa201fc9dfa772b005755214b6b96a27d4d7a2f9 Mon Sep 17 00:00:00 2001 From: Jeff Johnston Date: Tue, 16 Jun 2009 15:55:06 +0000 Subject: 2009-06-16 Craig Howland * libc/include/math.h: Simplify fpclassify, isinf, isnan, and signbit macros to remove un-necessary extension use. isinf and isnan also changed to use fpclassify. isfinite macro modified to run faster by only calling fpclassify once instead of possibly twice. --- newlib/ChangeLog | 7 +++++++ newlib/libc/include/math.h | 26 +++++++++++--------------- 2 files changed, 18 insertions(+), 15 deletions(-) diff --git a/newlib/ChangeLog b/newlib/ChangeLog index d501fe6..ef829ef 100644 --- a/newlib/ChangeLog +++ b/newlib/ChangeLog @@ -1,3 +1,10 @@ +2009-06-16 Craig Howland + + * libc/include/math.h: Simplify fpclassify, isinf, isnan, and signbit + macros to remove un-necessary extension use. isinf and isnan also + changed to use fpclassify. isfinite macro modified to run faster by + only calling fpclassify once instead of possibly twice. + 2009-06-09 Corinna Vinschen * libc/ctype/tolower.c (tolower): Cast conversion result from diff --git a/newlib/libc/include/math.h b/newlib/libc/include/math.h index f25539a..ca65a44 100644 --- a/newlib/libc/include/math.h +++ b/newlib/libc/include/math.h @@ -181,14 +181,14 @@ extern int __fpclassifyd (double x); extern int __signbitf (float x); extern int __signbitd (double x); -#define fpclassify(x) \ - (__extension__ ({__typeof__(x) __x = (x); \ - (sizeof (__x) == sizeof (float)) ? __fpclassifyf(__x) : __fpclassifyd(__x);})) +#define fpclassify(__x) \ + ((sizeof(__x) == sizeof(float)) ? __fpclassifyf(__x) : \ + __fpclassifyd(__x)) #ifndef isfinite -#define isfinite(y) \ - (__extension__ ({__typeof__(y) __y = (y); \ - fpclassify(__y) != FP_INFINITE && fpclassify(__y) != FP_NAN;})) + #define isfinite(__y) \ + (__extension__ ({int __cy = fpclassify(__y); \ + __cy != FP_INFINITE && __cy != FP_NAN;})) #endif /* Note: isinf and isnan were once functions in newlib that took double @@ -197,21 +197,17 @@ extern int __signbitd (double x); * now defined as macros. Implementations of the old functions * taking double arguments still exist for compatibility purposes. */ #ifndef isinf -#define isinf(x) \ - (__extension__ ({__typeof__(x) __x = (x); \ - (sizeof (__x) == sizeof (float)) ? __isinff(__x) : __isinfd(__x);})) + #define isinf(y) (fpclassify(y) == FP_INFINITE) #endif #ifndef isnan -#define isnan(x) \ - (__extension__ ({__typeof__(x) __x = (x); \ - (sizeof (__x) == sizeof (float)) ? __isnanf(__x) : __isnand(__x);})) + #define isnan(y) (fpclassify(y) == FP_NAN) #endif #define isnormal(y) (fpclassify(y) == FP_NORMAL) -#define signbit(x) \ - (__extension__ ({__typeof__(x) __x = (x); \ - (sizeof(__x) == sizeof(float)) ? __signbitf(__x) : __signbitd(__x);})) +#define signbit(__x) \ + ((sizeof(__x) == sizeof(float)) ? __signbitf(__x) : \ + __signbitd(__x)) #define isgreater(x,y) \ (__extension__ ({__typeof__(x) __x = (x); __typeof__(y) __y = (y); \ -- cgit v1.1