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. -- Måns Rullgård [email protected] _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
