[PATCH 2/4] [Powerpc] tune/optimize memmove/wordcopy. Remove switch statement.

Ryan S. Arnold ryan.arnold@gmail.com
Thu Mar 22 14:51:00 GMT 2012


On Mon, Mar 19, 2012 at 4:01 PM, Will Schmidt <will_schmidt@vnet.ibm.com> wrote:
> 2012-03-19  Will Schmidt <will_schmidt@vnet.ibm.com>
>
>        * sysdeps/powerpc/powerpc64/power6/wordcopy.c: Remove switch statement.
>        * sysdeps/powerpc/powerpc32/power6/wordcopy.c: Likewise.
> ---
>  sysdeps/powerpc/powerpc32/power6/wordcopy.c |   26 +-------------
>  sysdeps/powerpc/powerpc64/power6/wordcopy.c |   50 +--------------------------
>  2 files changed, 4 insertions(+), 72 deletions(-)
>
> diff --git a/sysdeps/powerpc/powerpc32/power6/wordcopy.c b/sysdeps/powerpc/powerpc32/power6/wordcopy.c
> index 9748268..af35e98 100644
> --- a/sysdeps/powerpc/powerpc32/power6/wordcopy.c
> +++ b/sysdeps/powerpc/powerpc32/power6/wordcopy.c
> @@ -116,18 +116,7 @@ _wordcopy_fwd_dest_aligned (dstp, srcp, len)
>     len -= 1;
>   }
>
> -  switch (align)
> -    {
> -    case 1:
> -      fwd_align_merge(1)
> -      break;
> -    case 2:
> -      fwd_align_merge(2)
> -      break;
> -    case 3:
> -      fwd_align_merge(3)
> -      break;
> -    }
> +  fwd_align_merge(align)

As with the previous patch this should have a space after the parenthesis, e.g.,

fwd_align_merge (align)

Ryan



More information about the Libc-alpha mailing list