Changeset 836

Show
Ignore:
Timestamp:
03/24/08 14:29:25 (8 months ago)
Author:
gbooker
Message:

Well, gcc assumes that we don't modify inputs to asm constraints, and we did. Fortunately, the input variables are not used later, so constraint changed to read/write, and we are fine. This shows up for width % 32 != 0, such as many of the 480p trailers

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved

  • trunk/ColorConversions.c

    r831 r836  
    255255                        "dec            %6                              \n\t" 
    256256                        "jnz            0b                              \n\t" 
    257                         : /*No outputs*/ 
    258                         : "r" (ov), "r" (ov2), 
    259                         "r" (yv), "r" (yv2), "r" (uv), "r" (vv), 
    260                         "r" (vWidth) 
     257                        : "+r" (ov), "+r" (ov2), 
     258                        "+r" (yv), "+r" (yv2), "+r" (uv), "+r" (vv) 
     259                        : "r" (vWidth) 
    261260                        ); 
    262261#else 
    …… 
    281280                } 
    282281#endif 
    283                  
    284                 for (x = vWidth * 16; x < halfwidth; x++) { 
     282 
     283                for (x=vWidth * 16; x < halfwidth; x++) { 
    285284                        unsigned x4 = x*4, x2 = x*2; 
    286285                        o2[x4] = o[x4] = uc[x];