[Zlib-devel] [PATCH 05/13] Add preprocessor define to tune Adler32 loop unrolling.

Arjan van de Ven arjanvandeven at gmail.com
Thu Nov 28 13:15:17 EST 2013


>
>> +#ifndef ADLER32_UNROLL_LESS
>>          while (len >= 16) {
>>              len -= 16;
>>              DO16(buf);
>>              buf += 16;
>> +#else
>> +        while (len >= 8) {
>> +            len -= 8;
>> +            DO8(buf, 0);
>> +            buf += 8;
>> +#endif
>
> can't this constant # be pulled out somehow so the whole body doesn't get
> duplicated ?


I looked at that.. but it's 3 lines only. When I tried to parameterize
it.. the code got a lot less readable than just duplicating these few
lines.




More information about the Zlib-devel mailing list