[PATCH, ARM] Fix minor issues in memchr NEON implementation

Corinna Vinschen vinschen@redhat.com
Wed Jun 7 10:16:00 GMT 2017


On Jun  2 11:22, Prakhar Bahuguna wrote:
> The NEON-optimised implementation of memchr has a couple of minor issues - it
> does not mask off the higher bits of the input character, and it uses a signed
> comparison instead of unsigned within the smallstr loop. This patch addresses
> both of these issues.
> 
> ChangeLog:
> 
> 2017-06-02  Prakhar Bahuguna  <prakhar.bahuguna@arm.com>
> 
> 	* libc/machine/arm/memchr.S: Add and instruction to mask off all but
> 	the lowest byte.
> 	Change blt to blo.
> 
> Okay for master?

Yes, pushed.


Thanks,
Corinna

-- 
Corinna Vinschen
Cygwin Maintainer
Red Hat
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://sourceware.org/pipermail/newlib/attachments/20170607/9621bf07/attachment.sig>


More information about the Newlib mailing list