This is the mail archive of the
mailing list for the glibc project.
Re: PowerPC LE strlen
- From: Will Schmidt <will_schmidt at vnet dot ibm dot com>
- To: "Ryan S. Arnold" <ryan dot arnold at gmail dot com>
- Cc: Alan Modra <amodra at gmail dot com>, libc-alpha <libc-alpha at sourceware dot org>, Adhemerval Zanella <azanella at linux dot vnet dot ibm dot com>
- Date: Wed, 14 Aug 2013 17:24:42 -0500
- Subject: Re: PowerPC LE strlen
- References: <20130809051815 dot GH3294 at bubble dot grove dot modra dot org> <CAAKybw9DqFeD8_ge2R2ZwYBPR6Lwx2TV_b_Qfi1-RFXtn6sR1w at mail dot gmail dot com>
- Reply-to: will_schmidt at vnet dot ibm dot com
On Tue, 2013-08-13 at 12:21 -0500, Ryan S. Arnold wrote:
> On Fri, Aug 9, 2013 at 12:18 AM, Alan Modra <firstname.lastname@example.org> wrote:
> > This is the first of nine patches adding little-endian support to
> > existing optimised string and memory functions. I did spend some
> > time with a power7 simulator looking at cycle by cycle behaviour for
> > memchr, but most of these patches have not been run on cpu
> > to check that we are going as fast as possible. I'm sure PowerPC
> > do better. However, the little-endian support mostly leaves main
> > loops unchanged, so I'm banking on previous authors having done a
> > good job on big-endian.. As with most code you stare at long
> > I found some improvements for big-endian too.
> I'd like Will Schmidt and Adhemerval Zanella to comment on this series
> of patches.
I've read through the rest of the set
(memset,memcpy,strchr,stpcpy,strcmp,strncmp,strlen), and don't have any
additional questions, though admit my eyes may have glazed over once or
The tweaks, adjustments, and improvements seem good. As long as they
have passed the tests, which I believe they have, I have no issues with
any of the changes.