Re: [PATCH] benchtests: Add more tests for memrchr

[Rajalakshmi Srinivasaraghavan <raji at linux dot vnet dot ibm dot com>]

On 05/24/2017 09:14 PM, H.J. Lu wrote:
>    FOR_EACH_IMPL (impl, 0)
>      do_one_test (impl, (CHAR *) (buf + align), seek_char, len, result);
> @@ -143,11 +144,27 @@ test_main (void)
>        do_test (i, 64, 256, 23);
>        do_test (0, 16 << i, 2048, 0);
>        do_test (i, 64, 256, 0);
> +#ifdef USE_AS_MEMRCHR
> +      /* Also test the position close to the beginning for memrchr.  */
> +      do_test (0, i, 256, 23);
> +      do_test (0, i, 256, 0);
> +      do_test (i, i, 256, 23);
> +      do_test (i, i, 256, 0);
> +#endif
>      }
>    for (i = 1; i < 32; ++i)
>      {
>        do_test (0, i, i + 1, 23);
>        do_test (0, i, i + 1, 0);
> +      do_test (i, i, i + 1, 23);
> +      do_test (i, i, i + 1, 0);
> +#ifdef USE_AS_MEMRCHR
> +      /* Also test the position close to the beginning for memrchr.  */
> +      do_test (0, 1, i + 1, 23);
> +      do_test (0, 2, i + 1, 0);
> +      do_test (i, i, i + 1, 23);
> +      do_test (i, i, i + 1, 0);

The above two lines are repeating, instead of
      do_test (i, 1, i + 1, 23);
      do_test (i, 2, i + 1, 0);

LGTM with the above change.

> +#endif
>      }
>
>    return ret;
> -- 2.9.4

--
Thanks
Rajalakshmi S