picnoir/glibc
1
0
Fork 0
Code Issues Pull requests Packages Projects Releases Wiki Activity

Further simplify power6 wordcopy by removing switch statements.

Browse source 
This fix replaces switch statements that contain individual
[fwd|bwd]_align_merge (<constant>) calls with a single [fwd|bwd]_align_merge
(align) call.
This commit is contained in:
Will Schmidt 2012-04-18 15:34:23 -05:00 committed by Ryan S. Arnold
parent b282631e36
commit 6b652f46c3
3 changed files with 13 additions and 72 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
ChangeLog
View file

@ -1,3 +1,12 @@
2012-04-18 Will Schmidt <will_schmidt@vnet.ibm.com>
* sysdeps/powerpc/powerpc64/power6/wordcopy.c
(_wordcopy_fwd_dest_aligned): Replace switch with a parameterized
fwd_align_merge macro call.
(_wordcopy_bwd_dest_aligned): Replace switch with a parameterized
bwd_align_merge macro call.
* sysdeps/powerpc/powerpc32/power6/wordcopy.c: Likewise.
2012-04-18 Will Schmidt <will_schmidt@vnet.ibm.com>
* sysdeps/powerpc/powerpc64/power6/wordcopy.c: Add fwd_align_merge and

26
sysdeps/powerpc/powerpc32/power6/wordcopy.c
View file

@ -115,18 +115,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);
}
@ -224,16 +213,5 @@ _wordcopy_bwd_dest_aligned (dstp, srcp, len)
len -= 1;
}
switch (align)
{
case 1:
bwd_align_merge (1);
break;
case 2:
bwd_align_merge (2);
break;
case 3:
bwd_align_merge (3);
break;
}
bwd_align_merge (align);
}

50
sysdeps/powerpc/powerpc64/power6/wordcopy.c
View file

@ -115,30 +115,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;
case 4:
fwd_align_merge (4);
break;
case 5:
fwd_align_merge (5);
break;
case 6:
fwd_align_merge (6);
break;
case 7:
fwd_align_merge (7);
break;
}
fwd_align_merge (align);
}
@ -236,28 +213,5 @@ _wordcopy_bwd_dest_aligned (dstp, srcp, len)
len -= 1;
}
switch (align)
{
case 1:
bwd_align_merge (1);
break;
case 2:
bwd_align_merge (2);
break;
case 3:
bwd_align_merge (3);
break;
case 4:
bwd_align_merge (4);
break;
case 5:
bwd_align_merge (5);
break;
case 6:
bwd_align_merge (6);
break;
case 7:
bwd_align_merge (7);
break;
}
bwd_align_merge (align);
}