Hi, 2011/7/28 Måns Rullgård <[email protected]>: > "Ronald S. Bultje" <[email protected]> writes: > >> Hi, >> >> On Thu, Jul 28, 2011 at 12:23 PM, Mans Rullgard <[email protected]> wrote: >>> Inspection of compiled code shows gcc handles these fine on its own. >>> Benchmarking also shows no measurable speed difference. >>> >>> Removing the remaining cases in get_cabac_bypass_sign_x86() does >>> cause more substantial changes to the compiled code with uncertain >>> impact. >>> >>> Signed-off-by: Mans Rullgard <[email protected]> >>> --- >>> libavcodec/x86/cabac.h | 12 +++--------- >>> libavcodec/x86/h264_i386.h | 32 +++++++------------------------- >>> 2 files changed, 10 insertions(+), 34 deletions(-) >> >> Fine with me. Does this conflict with Jason's atom patch? > > If it does, it should be trivial to merge. This is mostly removing lines.
OK then. Ronald _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
