diff options
author | H.J. Lu <hjl.tools@gmail.com> | 2016-01-14 16:35:24 -0800 |
---|---|---|
committer | H.J. Lu <hjl.tools@gmail.com> | 2016-01-14 16:35:40 -0800 |
commit | 82c9a4f85e8522bc9e578725075d5c0535940b32 (patch) | |
tree | a62d943911c77cdf16ea58ce2edcbf56c0cbe1ed | |
parent | d7890e6947114785755ae5b1cf5310491092ee0b (diff) | |
download | glibc-82c9a4f85e8522bc9e578725075d5c0535940b32.zip glibc-82c9a4f85e8522bc9e578725075d5c0535940b32.tar.gz glibc-82c9a4f85e8522bc9e578725075d5c0535940b32.tar.bz2 |
Use TIME_T_MAX and TIME_T_MIN in tst-mktime2.c
GCC 5.3 compiles
for (time_t_max = 1; 0 < time_t_max; time_t_max *= 2)
continue;
into an infinite loop with -Os. We can copy TIME_T_MAX and TIME_T_MIN
from time/mktime.c.
[BZ #19466]
* time/tst-mktime2.c (time_t_max): Removed.
(time_t_min): Likewise.
(TYPE_SIGNED): New.
(TYPE_MINIMUM): Likewise.
(TYPE_MAXIMUM): Likewise.
(TIME_T_MIN): Likewise.
(TIME_T_MAX): Likewise.
(mktime_test): Replace time_t_max and time_t_min with TIME_T_MAX
and TIME_T_MIN.
(do_test): Likewise.
-rw-r--r-- | ChangeLog | 14 | ||||
-rw-r--r-- | time/tst-mktime2.c | 38 |
2 files changed, 40 insertions, 12 deletions
@@ -1,3 +1,17 @@ +2016-01-14 H.J. Lu <hongjiu.lu@intel.com> + + [BZ #19466] + * time/tst-mktime2.c (time_t_max): Removed. + (time_t_min): Likewise. + (TYPE_SIGNED): New. + (TYPE_MINIMUM): Likewise. + (TYPE_MAXIMUM): Likewise. + (TIME_T_MIN): Likewise. + (TIME_T_MAX): Likewise. + (mktime_test): Replace time_t_max and time_t_min with TIME_T_MAX + and TIME_T_MIN. + (do_test): Likewise. + 2016-01-14 Amit Pawar <amit.pawar@amd.com> [BZ #19467] diff --git a/time/tst-mktime2.c b/time/tst-mktime2.c index bc7cc58..c41db3e 100644 --- a/time/tst-mktime2.c +++ b/time/tst-mktime2.c @@ -5,8 +5,28 @@ #include <stdlib.h> #include <unistd.h> -static time_t time_t_max; -static time_t time_t_min; +/* True if the arithmetic type T is signed. */ +#define TYPE_SIGNED(t) (! ((t) 0 < (t) -1)) + +/* The maximum and minimum values for the integer type T. These + macros have undefined behavior if T is signed and has padding bits. + If this is a problem for you, please let us know how to fix it for + your host. */ +#define TYPE_MINIMUM(t) \ + ((t) (! TYPE_SIGNED (t) \ + ? (t) 0 \ + : ~ TYPE_MAXIMUM (t))) +#define TYPE_MAXIMUM(t) \ + ((t) (! TYPE_SIGNED (t) \ + ? (t) -1 \ + : ((((t) 1 << (sizeof (t) * CHAR_BIT - 2)) - 1) * 2 + 1))) + +#ifndef TIME_T_MIN +# define TIME_T_MIN TYPE_MINIMUM (time_t) +#endif +#ifndef TIME_T_MAX +# define TIME_T_MAX TYPE_MAXIMUM (time_t) +#endif /* Values we'll use to set the TZ environment variable. */ static const char *tz_strings[] = @@ -53,8 +73,8 @@ static void mktime_test (time_t now) { mktime_test1 (now); - mktime_test1 ((time_t) (time_t_max - now)); - mktime_test1 ((time_t) (time_t_min + now)); + mktime_test1 ((time_t) (TIME_T_MAX - now)); + mktime_test1 ((time_t) (TIME_T_MIN + now)); } static void @@ -113,19 +133,13 @@ do_test (void) isn't worth using anyway. */ alarm (60); - for (time_t_max = 1; 0 < time_t_max; time_t_max *= 2) - continue; - time_t_max--; - if ((time_t) -1 < 0) - for (time_t_min = -1; (time_t) (time_t_min * 2) < 0; time_t_min *= 2) - continue; - delta = time_t_max / 997; /* a suitable prime number */ + delta = TIME_T_MAX / 997; /* a suitable prime number */ for (i = 0; i < N_STRINGS; i++) { if (tz_strings[i]) setenv ("TZ", tz_strings[i], 1); - for (t = 0; t <= time_t_max - delta; t += delta) + for (t = 0; t <= TIME_T_MAX - delta; t += delta) mktime_test (t); mktime_test ((time_t) 1); mktime_test ((time_t) (60 * 60)); |