diff mbox

benchtests/bench-strtod.c: Increase timeout value

Message ID 1395763999-28618-1-git-send-email-will.newton@linaro.org
State Accepted
Headers show

Commit Message

Will Newton March 25, 2014, 4:13 p.m. UTC
This benchmark can take longer than the default 2 seconds on slower
platforms, so increase it to 10 seconds.

ChangeLog:

2014-03-25  Will Newton <will.newton@linaro.org>

	* benchtests/bench-strtod.c (TIMEOUT): Define to 10.
---
 benchtests/bench-strtod.c | 3 +++
 1 file changed, 3 insertions(+)

Comments

Siddhesh Poyarekar March 25, 2014, 4:25 p.m. UTC | #1
On Tue, Mar 25, 2014 at 04:13:19PM +0000, Will Newton wrote:
> This benchmark can take longer than the default 2 seconds on slower
> platforms, so increase it to 10 seconds.
> 
> ChangeLog:
> 
> 2014-03-25  Will Newton <will.newton@linaro.org>
> 
> 	* benchtests/bench-strtod.c (TIMEOUT): Define to 10.
> ---
>  benchtests/bench-strtod.c | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/benchtests/bench-strtod.c b/benchtests/bench-strtod.c
> index fbe8040..d0c4922 100644
> --- a/benchtests/bench-strtod.c
> +++ b/benchtests/bench-strtod.c
> @@ -26,6 +26,9 @@
>  #undef INNER_LOOP_ITERS
>  #define INNER_LOOP_ITERS 65536
>  
> +/* On slower platforms this test needs more than the default 2 seconds.  */
> +#define TIMEOUT 10
> +
>  static const char *inputs[] =
>  {
>    "1e308",

The usual practice seems to be to put the TIMEOUT macro definition
just before including test-skeleton.c, but that's a minor detail.  The
change looks OK to me.

Thanks,
Siddhesh
diff mbox

Patch

diff --git a/benchtests/bench-strtod.c b/benchtests/bench-strtod.c
index fbe8040..d0c4922 100644
--- a/benchtests/bench-strtod.c
+++ b/benchtests/bench-strtod.c
@@ -26,6 +26,9 @@ 
 #undef INNER_LOOP_ITERS
 #define INNER_LOOP_ITERS 65536
 
+/* On slower platforms this test needs more than the default 2 seconds.  */
+#define TIMEOUT 10
+
 static const char *inputs[] =
 {
   "1e308",