[Zlib-devel] [PATCH 12/13] deflate: add new deflate_quick strategy for level 1

[Jim Kukunas]

On Thu, Nov 28, 2013 at 02:05:05AM -0500, Mike Frysinger wrote:
> On Monday 25 November 2013 17:21:48 Jim Kukunas wrote:
> > --- a/configure
> > +++ b/configure
> > @@ -817,6 +817,9 @@ case "${ARCH}" in
> >              CRC_FOLDING_o=""
> >              CRC_FOLDING_lo=""
> >          fi
> > +
> > +        CFLAGS="${CFLAGS} -DUSE_QUICK"
> > +        SFLAGS="${SFLAGS} -DUSE_QUICK"
> >      ;;
> >      i386 | i486 | i586 | i686)
> >          OBJC="${OBJC} x86.o"
> > @@ -854,6 +857,9 @@ case "${ARCH}" in
> >              CRC_FOLDING_o=""
> >              CRC_FOLDING_lo=""
> >          fi
> > +
> > +        CFLAGS="${CFLAGS} -DUSE_QUICK"
> > +        SFLAGS="${SFLAGS} -DUSE_QUICK"
> >      ;;
> >  esac
> 
> unless i followed the diffs wrong, it seems like the end result is that x86_64 
> and i?86 branches are exactly the same.  what was the point of keeping them 
> separate ?

Hi Mike,

Currently, there is one difference between the x86_64 and i*86
branches: the i*86 branch defines CHECK_SSE2, whereas the x86_64 branch
doesn't, as SSE2 is part of the x86_64 spec.

I kept these separate because if you're changing something here, you
really should verify it for both separately.

Thanks.

-- 
Jim Kukunas
Intel Open Source Technology Center
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: not available
URL: <http://madler.net/pipermail/zlib-devel_madler.net/attachments/20131129/61e5f0f7/attachment-0001.sig>