diff mbox series

benchtests: Remove duplicated loop in bench-bzero-walk.c

Message ID 20220225032436.2525935-1-hjl.tools@gmail.com
State Committed
Commit cf92721befb557cecc82a862d3ac8cb6b47d4635
Headers
Series benchtests: Remove duplicated loop in bench-bzero-walk.c |

Checks

Context Check Description
dj/TryBot-apply_patch success Patch applied to master at the time it was sent
dj/TryBot-32bit success Build for i686

Commit Message

H.J. Lu Feb. 25, 2022, 3:24 a.m. UTC 
  Remove one of 2 identical loops in bench-bzero-walk.c.
---
 benchtests/bench-bzero-walk.c | 6 ------
 1 file changed, 6 deletions(-)

Comments

Siddhesh Poyarekar Feb. 25, 2022, 5:31 a.m. UTC | #1 
On 25/02/2022 08:54, H.J. Lu via Libc-alpha wrote:
> Remove one of 2 identical loops in bench-bzero-walk.c.

LGTM.

Reviewed-by: Siddhesh Poyarekar <siddhesh@sourceware.org>

> ---
>   benchtests/bench-bzero-walk.c | 6 ------
>   1 file changed, 6 deletions(-)
> 
> diff --git a/benchtests/bench-bzero-walk.c b/benchtests/bench-bzero-walk.c
> index 4984f6e190..851a72b137 100644
> --- a/benchtests/bench-bzero-walk.c
> +++ b/benchtests/bench-bzero-walk.c
> @@ -122,12 +122,6 @@ test_main (void)
>         do_test (&json_ctx, i + 1);
>       }
>   
> -  for (i = START_SIZE; i <= MIN_PAGE_SIZE; i <<= 1)
> -    {
> -      do_test (&json_ctx, i);
> -      do_test (&json_ctx, i + 1);
> -    }
> -
>     json_array_end (&json_ctx);
>     json_attr_object_end (&json_ctx);
>     json_attr_object_end (&json_ctx);
diff mbox series

Patch

diff --git a/benchtests/bench-bzero-walk.c b/benchtests/bench-bzero-walk.c
index 4984f6e190..851a72b137 100644
--- a/benchtests/bench-bzero-walk.c
+++ b/benchtests/bench-bzero-walk.c
@@ -122,12 +122,6 @@  test_main (void)
       do_test (&json_ctx, i + 1);
     }
 
-  for (i = START_SIZE; i <= MIN_PAGE_SIZE; i <<= 1)
-    {
-      do_test (&json_ctx, i);
-      do_test (&json_ctx, i + 1);
-    }
-
   json_array_end (&json_ctx);
   json_attr_object_end (&json_ctx);
   json_attr_object_end (&json_ctx);