[PATCH 4/4] arm: Enable ARM mode for armv6 strlen

Phil Blundell pb@pbcl.net
Wed Apr 11 22:12:00 GMT 2018


On Wed, 2018-04-11 at 18:16 -0300, Adhemerval Zanella wrote:
> Current optimized armv6t2 strlen uses the NO_THUMB wrongly to
> conditionalize thumb instruction usage.  The flags is meant to be
> defined before sysdep.h inclusion and to indicate the assembly
> requires to build in ARM mode, not to check whether thumb is
> enable or not.  This patch fixes it by using the GCC provided
> '__thumb__' instead.

Is it ever useful to build for ARM-state when on armv6t2 (i.e. Thumb2
is guaranteed to be available)?  It's not totally obvious from reading
the source that the ARM version is going to be better in any way than
the Thumb one.

Assuming that this is indeed something worth supporting, I think the
short log for the patch ought to say "armv6t2" not "armv6".

p.



More information about the Libc-alpha mailing list