This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: [PATCH 2/4] [Powerpc] tune/optimize memmove/wordcopy. Removeswitch statement.
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