[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