From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from gate.crashing.org (gate.crashing.org [63.228.1.57]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) by ozlabs.org (Postfix) with ESMTP id 769C1DDF1C for ; Wed, 5 Mar 2008 16:26:18 +1100 (EST) In-Reply-To: <18382.7074.69060.661720@cargo.ozlabs.ibm.com> References: <1204301097.14759.6.camel@localhost.localdomain> <1204340690.15052.457.camel@pasglop> <18382.7074.69060.661720@cargo.ozlabs.ibm.com> Mime-Version: 1.0 (Apple Message framework v623) Content-Type: text/plain; charset=US-ASCII; format=flowed Message-Id: From: Segher Boessenkool Subject: Re: [PATCH] add strncmp to PowerPC Date: Wed, 5 Mar 2008 06:26:06 +0100 To: Paul Mackerras Cc: LKML , Steven Rostedt , linuxppc-dev@ozlabs.org List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , >> Do we have any indication that it performs better than the C one ? > > I would expect it to, given that the assembler one has two branches in > the per-byte loop compared to 3 in the C version. But really, does it matter for strncmp() in the kernel? Anyway, this asm code has bugs, as do both the current C version in the kernel, and the code I posted. We need to do better :-) Segher