Re: [PATCH] Makefile: Add '-fno-builtin-bcmp' to CLANG_FLAGS
From: Nick Desaulniers
Date: Wed Mar 13 2019 - 13:22:01 EST
On Wed, Mar 13, 2019 at 8:32 AM Nathan Chancellor
> On Wed, Mar 13, 2019 at 02:48:49PM +0100, Arnd Bergmann wrote:
> > On Wed, Mar 13, 2019 at 2:44 PM Nathan Chancellor
> > <natechancellor@xxxxxxxxx> wrote:
> > > On Wed, Mar 13, 2019 at 09:13:11AM +0100, Rasmus Villemoes wrote:
> > > > Wouldn't it be better to just define bcmp as an alias for memcmp? They
> > > > seem to have compatible prototypes, and then somebody might someday sit
> > > > down and implement some word-at-a-time version of bcmp making use of the
> > > > weaker guarantees about the return value to gain some performance. But I
> > > > suppose that can also be done later.
> > >
> > > Thank you much for the review, I didn't even realize this was possible :)
> > >
> > > I'd certainly like to explore it as that is what glibc does. How would
> > > you suggest going about it here?
> > I suggested a possible implementation (likely contains bugs) and
> > an alias for architectures that require strict alignment, see
> > https://bugs.llvm.org/show_bug.cgi?id=41035#c11
> > We could start out with just the alias.
> > Arnd
> So I've been messing around with this for a bit (forgive me, I'm still
> learning all of the intricacies around here) and this is what I came up
> with for when __ARCH_HAVE_MEMCMP is unset (not particularly difficult
> obviously). I can compile, link, and boot an x86 in QEMU.
> However, I am not sure how to handle memcmp definitions that are written
> in assembly like arm64, as the alias attribute only works when the alias
> is defined in the same translation unit. Thoughts?
I hit this, too:
./arch/arm64/include/asm/string.h:40:15: error: alias must point to a
since memcmp is only declared (not defined) in that header, clang is
not happy to alias to memcmp. If __HAVE_ARCH_MEMCMP is defined, then
we can just return a call to memcmp. Thoughts (I need to add comments
above bcmp, anything else)? Do we like the typeof trick (stolen from
glibc) or no?
diff --git a/lib/string.c b/lib/string.c
index 38e4ca08e757..e6c1954f2716 100644
@@ -845,7 +845,13 @@ void *memmove(void *dest, const void *src, size_t count)
+int bcmp(const void *cs, const void *ct, size_t n)
+ return memcmp(cs, ct, n);
* memcmp - Compare two areas of memory
* @cs: One area of memory
@@ -864,6 +870,8 @@ __visible int memcmp(const void *cs, const void
*ct, size_t count)
+__weak __alias(memcmp) typeof(memcmp) bcmp;