From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <libc-alpha-return-63507-listarch-libc-alpha=sources.redhat.com@sourceware.org>
Received: (qmail 36505 invoked by alias); 26 Sep 2015 08:46:04 -0000
Mailing-List: contact libc-alpha-help@sourceware.org; run by ezmlm
Precedence: bulk
List-Id: <libc-alpha.sourceware.org>
List-Subscribe: <mailto:libc-alpha-subscribe@sourceware.org>
List-Archive: <http://sourceware.org/ml/libc-alpha/>
List-Post: <mailto:libc-alpha@sourceware.org>
List-Help: <mailto:libc-alpha-help@sourceware.org>, <http://sourceware.org/ml/#faqs>
Sender: libc-alpha-owner@sourceware.org
Received: (qmail 36493 invoked by uid 89); 26 Sep 2015 08:46:04 -0000
Authentication-Results: sourceware.org; auth=none
X-Virus-Found: No
X-Spam-SWARE-Status: No, score=-0.7 required=5.0 tests=AWL,BAYES_00,FREEMAIL_FROM,SPF_NEUTRAL autolearn=no version=3.3.2
X-HELO: popelka.ms.mff.cuni.cz
Date: Sat, 26 Sep 2015 08:46:00 -0000
From: =?utf-8?B?T25kxZllaiBCw61sa2E=?= <neleai@seznam.cz>
To: Wilco Dijkstra <wdijkstr@arm.com>
Cc: 'GNU C Library' <libc-alpha@sourceware.org>
Subject: Re: [PATCH][AArch64] Add optimized memchr
Message-ID: <20150926084544.GA31280@domone>
References: <002d01d0f795$0ce77eb0$26b67c10$@com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <002d01d0f795$0ce77eb0$26b67c10$@com>
User-Agent: Mutt/1.5.20 (2009-06-14)
X-SW-Source: 2015-09/txt/msg00661.txt.bz2

On Fri, Sep 25, 2015 at 02:21:13PM +0100, Wilco Dijkstra wrote:
> An optimized memchr was missing for AArch64. This version is similar to strchr and is significantly
> faster than the C version. Passes GLIBC tests.
> 
> OK for commit?
> 
> ChangeLog:
> 2015-09-25  Wilco Dijkstra  <wdijkstr@arm.com>
> 2015-09-25  Kevin Petit  <kevin.petit@arm.com>
> 
> 	* sysdeps/aarch64/memchr.S (__memchr): New file.

How you tested performance. I think that also here loading first 32
bytes unaligned should be better. Could you use dryrun to verify?

Also same optimization could be used for memrchr.