On Mon, Jun 20, 2011 at 09:05:50AM +0100, Måns Rullgård wrote: > Diego Biurrun <[email protected]> writes: > > > On Mon, Jun 20, 2011 at 02:40:46AM +0100, Mans Rullgard wrote: > >> > >> --- a/libavcodec/cabac.h > >> +++ b/libavcodec/cabac.h > >> @@ -307,17 +307,6 @@ static inline void > >> renorm_cabac_decoder_once(CABACContext *c){ > >> > >> static av_always_inline int get_cabac_inline(CABACContext *c, uint8_t * > >> const state){ > >> //FIXME gcc generates duplicate load/stores for c->low and c->range > >> -#define LOW "0" > >> -#define RANGE "4" > >> -#if ARCH_X86_64 > >> -#define BYTESTART "16" > >> -#define BYTE "24" > >> -#define BYTEEND "32" > >> -#else > >> -#define BYTESTART "12" > >> -#define BYTE "16" > >> -#define BYTEEND "20" > >> -#endif > > > > The FIXME line should be removed as well. > > Why? My patch does not change the validity of that FIXME. It might be > pointless already, of course.
Yes, scratch my comment. Diego _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
