# HG changeset patch # User arpi_esp # Date 1010982892 0 # Node ID fceb435fae6b2c1aa3e1219a7a219eb728273447 # Parent 9bfd3abd85ebe4b35d4352c7d0d952a1a7ee38fe (commit by michael) 16-bit divide instead of 32-bit on x86 in msmpeg_pred_dc() diff -r 9bfd3abd85eb -r fceb435fae6b msmpeg4.c --- a/msmpeg4.c Mon Jan 14 04:29:19 2002 +0000 +++ b/msmpeg4.c Mon Jan 14 04:34:52 2002 +0000 @@ -411,10 +411,34 @@ necessitate to modify mpegvideo.c. The problem comes from the fact they decided to store the quantized DC (which would lead to problems if Q could vary !) */ +#ifdef ARCH_X86 + /* using 16bit divisions as they are large enough and 2x as fast */ + asm volatile( + "movl %3, %%eax \n\t" + "shrl $1, %%eax \n\t" + "addl %%eax, %2 \n\t" + "addl %%eax, %1 \n\t" + "addl %0, %%eax \n\t" + "xorl %%edx, %%edx \n\t" + "divw %w3 \n\t" + "movzwl %%ax, %0 \n\t" + "movl %1, %%eax \n\t" + "xorl %%edx, %%edx \n\t" + "divw %w3 \n\t" + "movzwl %%ax, %1 \n\t" + "movl %2, %%eax \n\t" + "xorl %%edx, %%edx \n\t" + "divw %w3 \n\t" + "movzwl %%ax, %2 \n\t" + : "+r" (a), "+r" (b), "+r" (c) + : "r" (scale) + : "%eax", "%edx" + ); +#else a = (a + (scale >> 1)) / scale; b = (b + (scale >> 1)) / scale; c = (c + (scale >> 1)) / scale; - +#endif /* XXX: WARNING: they did not choose the same test as MPEG4. This is very important ! */ if (abs(a - b) <= abs(b - c)) {