Andrew Pinski <[email protected]> writes:

> 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.

Sorry, this one fell off the stack. Will handle it tonight.

>
> 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