[PATCH] PowerPC: memmove default implementation cleanup
Adhemerval Zanella
azanella@linux.vnet.ibm.com
Tue Jun 24 18:35:00 GMT 2014
On 24-06-2014 13:11, OndÅej BÃlka wrote:
> On Tue, Jun 24, 2014 at 11:21:30AM -0300, Adhemerval Zanella wrote:
>> This patch removes the powerpc specific logic in memmove and instead
>> include default implementation with MEMCPY_OK_FOR_FWD_MEMMOVE defined.
>> This lead in a increase performance, since the constraints to use
>> memcpy in powerpc code are too restrictive and memcpy can be used for
>> any forward memmove (and the rest to PPC implementation is just a copy
>> of default one).
>>
>> Checked on powerpc32-fpu and powerpc64be. OK to apply?
>>
> That looks ok, I would go step more and say that aliasing memcpy to
> memmove will differ only by few cycles and could improve performance
> due of cache locality but I did not do test yet.
>
Thanks for review. Now with optimized power7, I think I can check if alias
shows a performance boost (although such few cycles will be hard to evaluate
due os jitter). However, I don't think we can enable now for all power
implementations (since for power4, power6, etc. that have optimized
memcpy, this will mean another branch).
More information about the Libc-alpha
mailing list