[Ffmpeg-cvslog] r7525 - trunk/libavutil/aes.c

[Michael Niedermayer]

Hi

On Mon, Jan 15, 2007 at 11:07:47AM +0100, Michel Bardiaux wrote:
> michael wrote:
> >Author: michael
> >Date: Mon Jan 15 02:01:56 2007
> >New Revision: 7525
> >
> >Modified:
> >   trunk/libavutil/aes.c
> >
> >Log:
> >replace / by >> this makes the .o ~150 bytes smaller
> 
> What are the version of gcc and the configure options? Its quite 
> astonishing, I would have expected any halfway decent compiler to 
> convert every division by a power of two, to shifts, so that this source 
> change would have no effect.

(-1)/2 != (-1)>>1 and the variables are signed, of course are they never
negative but gcc is too stupid to realize that so gcc cannot use a plain
shift

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

No evil is honorable: but death is honorable; therefore death is not evil.
-- Citium Zeno
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-cvslog/attachments/20070115/ab22b06a/attachment.pgp>