Hi, On Sat, Mar 24, 2012 at 11:17 PM, Alexander Strange <[email protected]> wrote: > On Mon, Mar 19, 2012 at 10:18 AM, Ronald S. Bultje <[email protected]> wrote: >> On Sat, Mar 17, 2012 at 9:34 AM, Ronald S. Bultje <[email protected]> wrote: >>> --- >>> libavcodec/x86/cabac.h | 10 +++++----- >>> libavcodec/x86/h264_i386.h | 32 +++++++++++++++++--------------- >>> 2 files changed, 22 insertions(+), 20 deletions(-) >> >> Ping for these 4 patches also (5/8-8/8), they touch a different part >> of the code and can thus be done separately. > > What is this for? I can see it making code density a bit worse > (compiler can't keep c->bytestream in a register between asm > statements).
Without this patch, some compilers (see other email) mess up register allocations and adding another "+m"(..) makes it run out of register and fail to compile. Yes, it's a compiler bug, but widespread enough that a workaround is warranted, especially since it also helps "bug-free" compilers generate better code (again, see other patch). Ronald _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
