On Mon, 1 Jan 2007, Mark Brown wrote:

> On Wed, Sep 06, 2006 at 05:51:39PM -0700, dean gaudet wrote:
> 
> > note that this define wasn't necessary on 32-bit x86 because there's 
> > custom 32-bit assembly which uses unaligneds even more aggressively than 
> > the C code does even when given UNALIGNED_OK.
> 
> Which custom 32 bit assembly are you referring to here?

my apologies... i usually research my bug reports better.

these files have assembly:

./build-tree/zlib-1.2.3/contrib/asm586/match.S
./build-tree/zlib-1.2.3/contrib/asm686/match.S
./build-tree/zlib-1.2.3/contrib/inflate86/inffast.S

but it doesn't appear that they're actually being used.

and i can't even reproduce my results... here's the averages of the user
cpu seconds for 10 runs of "minizip -9o a.zip linux-2.6.19.tar":

         baseline  -DUNALIGNED_OK
k8 revF    26.62       26.59
core2      28.43       28.44

the differences are measurement noise... huh.

and similarly for miniunz:

         baseline  -DUNALIGNED_OK
k8 revF     1.29        1.30
core2       1.47        1.49

i wonder what i did differently the day i filed that report... i know
i saw an improvement that time :)

sorry for wasting your time... go ahead and close this out (unless you 
want to use it as a reminder to see if the 32-bit assembly helps...)

-dean


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to