[FFmpeg-devel] [PATCH 1/5] Shrink the size of vp56_mv

[Michael Niedermayer]

On Tue, Jun 15, 2010 at 08:09:46PM +0100, M?ns Rullg?rd wrote:
> Reimar D?ffinger <Reimar.Doeffinger at gmx.de> writes:
> 
> > On Tue, Jun 15, 2010 at 04:58:14PM +0200, Aurelien Jacobs wrote:
> >> On Tue, Jun 15, 2010 at 09:02:14AM -0400, David Conrad wrote:
> >> > ---
> >> >  libavcodec/vp56.h |    4 ++--
> >> >  1 files changed, 2 insertions(+), 2 deletions(-)
> >> > 
> >> > diff --git a/libavcodec/vp56.h b/libavcodec/vp56.h
> >> > index 89eba05..3d98b9e 100644
> >> > --- a/libavcodec/vp56.h
> >> > +++ b/libavcodec/vp56.h
> >> > @@ -60,8 +60,8 @@ typedef struct {
> >> >  } VP56RefDc;
> >> >  
> >> >  struct vp56_mv {
> >> > -    int x;
> >> > -    int y;
> >> > +    int16_t x;
> >> > +    int16_t y;
> >> >  };
> >> 
> >> Looks OK. Apply if you're sure that it can't overflow.
> >
> > Hm, might make sense to benchmark a bit, it might be slower e.g.
> > if it means a lot of sign extension is added to the code...
> 
> What CPU can't load signed half-words from memory?

with gcc id be happy if it managed to load ints from memory without
unneeded sign/zero extensions on x86_64

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Awnsering whenever a program halts or runs forever is
On a turing machine, in general impossible (turings halting problem).
On any real computer, always possible as a real computer has a finite number
of states N, and will either halt in less than N cycles or never halt.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100615/4e9e6502/attachment.pgp>