Re: memchr.c and memrchr.c portability fixes

[Andreas Schwab]

Paul Eggert <address@hidden> writes:

>    /* Handle the first few characters by reading one character at a time.
>       Do this until CHAR_PTR is aligned on a longword boundary.  */
>    for (char_ptr = (const unsigned char *) s;
> -       n > 0 && ((unsigned long int) char_ptr
> -              & (sizeof (longword) - 1)) != 0;
> +       n > 0 && UNALIGNED_P (char_ptr);

This means that char_ptr may no longer be aligned to longword boundary
after the loop, which will degrade performance on some hosts.  For
example, on m68k-linux alignof(type) will always be at most 2, but you
get better performance with a 4-byte aligned pointer.

Andreas.

-- 
Andreas Schwab, SuSE Labs, address@hidden
SuSE Linux AG, Maxfeldstraße 5, 90409 Nürnberg, Germany
Key fingerprint = 58CA 54C7 6D53 942B 1756  01D3 44D5 214B 8276 4ED5
"And now for something completely different."