From mboxrd@z Thu Jan  1 00:00:00 1970
From: Chris Moore <moore@free.fr>
Date: Thu, 08 Oct 2009 07:23:05 +0200
Subject: [U-Boot] [PATCH 1/3] memcpy: use 32-bit copies if possible
In-Reply-To: <b03f28d4c4afa0c5c9f2421b7d260a0f3890cdcd.1254904388.git.rubini@	unipv.it>
References: <cover.1254904388.git.rubini@unipv.it>
	<b03f28d4c4afa0c5c9f2421b7d260a0f3890cdcd.1254904388.git.rubini@	unipv.it>
Message-ID: <4ACD7739.8040800@free.fr>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Alessandro Rubini a ?crit :
> From: Alessandro Rubini <rubini@unipv.it>
>
> Signed-off-by: Alessandro Rubini <rubini@unipv.it>
> Acked-by: Andrea Gallo <andrea.gallo@stericsson.com>
> ---
>  lib_generic/string.c |   11 ++++++++++-
>  1 files changed, 10 insertions(+), 1 deletions(-)
>
> diff --git a/lib_generic/string.c b/lib_generic/string.c
> index 181eda6..fdccab6 100644
> --- a/lib_generic/string.c
> +++ b/lib_generic/string.c
> @@ -449,7 +449,16 @@ char * bcopy(const char * src, char * dest, int count)
>  void * memcpy(void * dest,const void *src,size_t count)
>  {
>  	char *tmp = (char *) dest, *s = (char *) src;
> +	u32 *d32 = (u32 *)dest, *s32 = (u32 *) src;
>  
> +	/* if both are aligned, use 32-bit copy */
> +	if ( (((int)dest & 3) | ((int)src & 3) | (count & 3)) == 0 ) {
>   

This can be factorized as (a & 3) | (b & 3) | (c & 3) is equivalent to 
(a | b | c) & 3.
GCC is pretty smart but I doubt that it will pick this up :(

> +		count /= 4;
> +		while (count--)
> +			*d32++ = *s32++;
> +		return dest;
> +	}
> +	/* else, use 1-byte copy */
>  	while (count--)
>  		*tmp++ = *s++;
>  
>   

Cheers,
Chris