Re: CVS commit: src/common/lib/libc/arch/arm/string

[Matt Thomas <matt%3am-software.com@localhost>]

On Jan 15, 2013, at 12:10 AM, David Laight wrote:

> On Tue, Jan 15, 2013 at 02:03:31AM +0000, Matt Thomas wrote:
>> Module Name: src
>> Committed By:        matt
>> Date:                Tue Jan 15 02:03:31 UTC 2013
>> 
>> Added Files:
>>      src/common/lib/libc/arch/arm/string: strchr_naive.S strrchr_naive.S
>> 
>> Log Message:
>> Add simple/small versions of strchr/strrchr for ARM.
> 
> If you use 'preindex with writeback' you can remove an instruction
> from the loop (in strchr at least).

I pretty much already do that.

> I'm also intrigued that these are the very simple/small versions
> whereas the memcpy() (I think) was one that did all the alignment
> and double-word copies!

As I said in its commit, the stupid version was very very slow.