Re: [PATCH] remove ARM and Mozilla SHA1 implementations

From: Johannes Schindelin <johannes.schindelin@gmx.de>
Date: Aug 24, 2009, 16:52 UTC
Message-ID: <alpine.DEB.1.00.0908241849160.8306@pacific.mpi-cbg.de>
In-Reply-To: <alpine.DEB.1.00.0908241333130.11375@intel-tinevez-2-302>

Hi,

On Mon, 24 Aug 2009, Johannes Schindelin wrote:

Show 19 quoted lines

> On Mon, 24 Aug 2009, Nicolas Pitre wrote:
> 
> > On Tue, 18 Aug 2009, Johannes Schindelin wrote:
> > 
> > > On Mon, 17 Aug 2009, Nicolas Pitre wrote:
> > > 
> > > > They are both slower than the new BLK_SHA1 implementation, so it is 
> > > > pointless to keep them around.
> > > > 
> > > > Signed-off-by: Nicolas Pitre <nico@cam.org> ---
> > > > 
> > > > Someone else would need to make the call for the PPC version.
> > > 
> > > If I don't forget, I can test tomorrow on 2 different 32-bit PPCs and 
> > > possibly one 64-bit PPC.
> > 
> > Did you forget?  ;-)
> 
> Yes.

At long last (XTools took a real long time to install here; it is an old PowerPC running MacOSX):

Best of 10 "git rev-list --all" runs on a full Git repository (including my own tree):

before 30ae47b4

	1.85 real, 1.52 user, 0.28 sys

after 30ae47b4

	1.86 real, 1.52 user, 0.28 sys

To be frank, the 1.85 looks like an outlier, so I think there is exactly 0 difference between the two.

If you want me to perform other tests, please let me know; I'll be back at this computer tomorrow.

Ciao, Dscho