[U-Boot] [PATCH] uboot optimize memmove

[Wolfgang Denk]

Dear Andy Green,

In message <CAAfg0W4wXdRafRcHvrNVNnkCGd3EvJfNujuzUsjQXgW=WmCU+w at mail.gmail.com> you wrote:
>
> Given that code should perferably never be used, maybe it should print
> a warning like "Using default memory ops" and leave it like it is.
> The problem is not correctness just inefficiency.

Agreed. You found a problem, and a way to improve it.  All this is
perfectly fine.  I just didn't like your implementation, and rather
suggest to copy the 32-bit-copy loop as used in the current memcpy()
code and use it also in memmove().

I'm looking forward to seeing your patch v2 with such an improved
implementation.  Thanks in advance.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
The biggest difference between time and space is that you can't reuse
time.                                                 - Merrick Furst