[haiku-commits] Re: haiku: hrev52253 - in src: system/libroot/posix/string/arch/arm data/package_infos/any data/package_infos data/package_infos/generic

  • From: Adrien Destugues <pulkomandy@xxxxxxxxxxxxx>
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Mon, 20 Aug 2018 18:26:47 +0200

On Mon, Aug 20, 2018 at 11:54:40AM -0400, waddlesplash wrote:

b2cb85f4d5aa: libroot: New memcpy for ARM.
  
  It seems the old one was just broken (see mailing list discussion.)
  
  This new one was built from the generic one, but it was done using
  a trunk build of Clang 8 with tuned optimization flags, and is
  smaller (101 instructions) than GCC 8's (134 instructions) and
  the old hand-optimized one (~125 instructions?) as well as being
  targeted for ARMv7 instead of ARMv6 like the old one was.

We don't have coding rules for asm, but at least naming the labels would
be nice. Or if this is compiled from something, why not include the C++
sources instead?

-- 
Adrien.

Other related posts: