On Sun, Mar 30, 2025 at 8:41 AM Sam James <[email protected]> wrote:
>
> Fixed the iteration number in crc-crc32-data16.c test from 8 to 16 to
> match the test name, just like in r15-9038-gdf55a933cfc675.
>
> gcc/testsuite/ChangeLog:
>         * gcc.target/aarch64/crc-crc32-data16.c: Fix iteration
>         count to match testname.
> ---
> Do we need this as well? Untested so far.

Yes please. I see it was approved by Jeff but never applied.

Thanks,
Andrew

>
>  gcc/testsuite/gcc.target/aarch64/crc-crc32-data16.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/gcc/testsuite/gcc.target/aarch64/crc-crc32-data16.c 
> b/gcc/testsuite/gcc.target/aarch64/crc-crc32-data16.c
> index e82cb04fcc38..7561b3ba5407 100644
> --- a/gcc/testsuite/gcc.target/aarch64/crc-crc32-data16.c
> +++ b/gcc/testsuite/gcc.target/aarch64/crc-crc32-data16.c
> @@ -10,7 +10,7 @@ uint32_t _crc32_O0 (uint32_t crc, uint16_t data) {
>    int i;
>    crc = crc ^ data;
>
> -  for (i = 0; i < 8; i++) {
> +  for (i = 0; i < 16; i++) {
>        if (crc & 1)
>         crc = (crc >> 1) ^ 0xEDB88320;
>        else
> @@ -24,7 +24,7 @@ uint32_t _crc32 (uint32_t crc, uint16_t data) {
>    int i;
>    crc = crc ^ data;
>
> -  for (i = 0; i < 8; i++) {
> +  for (i = 0; i < 16; i++) {
>        if (crc & 1)
>         crc = (crc >> 1) ^ 0xEDB88320;
>        else
>
> base-commit: df55a933cfc675be2024b16386b96b2807464b41
> --
> 2.49.0
>

Reply via email to