Panagiotis Issaris <[EMAIL PROTECTED]> added the comment:

Diego Biurrun schreef:
> Diego Biurrun <[EMAIL PROTECTED]> added the comment:
> 
> On Wed, Dec 12, 2007 at 08:51:52AM +0000, Panagiotis Issaris wrote:
>> Panagiotis Issaris <[EMAIL PROTECTED]> added the comment:
>>
>> Diego Biurrun schreef:
>>> Diego Biurrun <[EMAIL PROTECTED]> added the comment:
>>>
>>> On Tue, Dec 11, 2007 at 06:46:07PM +0000, Diego Biurrun wrote:
>>>> Diego Biurrun <[EMAIL PROTECTED]> added the comment:
>>>>
>>>> On Tue, Dec 11, 2007 at 12:52:57PM +0000, Panagiotis Issaris wrote:
>>>>> Panagiotis Issaris <[EMAIL PROTECTED]> added the comment:
>>>>>
>>>>> Okay, I just verified that --disable-encoder=flac doesn't avoid the 
>>>>> problem as
>>>>> the flac SSE2 code gets compiled anyway.
>>>> This is a bug, fix it :)
>>> Please try the attached patch.
>> Are you sure about this patch? Disabling FLAC SSE when the user disables
>> CAVS? CAVS and FLAC are unrelated so I'd prefer having two separate
>> solutions for the two problems.
> 
> Umm, sorry, that's obviously a typo.  Please replace CAVS by FLAC and
> let me know if it works or not.

No, it doesn't work as flac_computer_autocorr_sse2 still gets compiled:

make[1]: Nothing to be done for `all'.
make -C libavcodec  all
gcc -I"/Users/takis/src/ffmpeg-rw"/libswscale
-I"/Users/takis/src/ffmpeg-rw"/libavcodec  -DHAVE_AV_CONFIG_H
-D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -D_ISOC9X_SOURCE
-I"/Users/takis/build/ffmpeg" -I"/Users/takis/src/ffmpeg-rw"
-I"/Users/takis/src/ffmpeg-rw"/libavutil -no-cpp-precomp -pipe
-force_cpusubtype_ALL -Wno-sign-compare -fomit-frame-pointer -g
-Wdeclaration-after-statement -Wall -Wno-switch -Wdisabled-optimization
-Wpointer-arith -Wredundant-decls -Wno-pointer-sign -O3  -c -o
i386/dsputil_mmx.o /Users/takis/src/ffmpeg-rw/libavcodec/i386/dsputil_mmx.c
/Users/takis/src/ffmpeg-rw/libavcodec/i386/dsputil_mmx.c: In function
'gmc_mmx':
/Users/takis/src/ffmpeg-rw/libavcodec/i386/dsputil_mmx.c:2669: warning:
suggest parentheses around arithmetic in operand of |
/Users/takis/src/ffmpeg-rw/libavcodec/i386/dsputil_mmx.c:2669: warning:
suggest parentheses around arithmetic in operand of |
/Users/takis/src/ffmpeg-rw/libavcodec/i386/dsputil_mmx.c:2670: warning:
suggest parentheses around arithmetic in operand of |
/Users/takis/src/ffmpeg-rw/libavcodec/i386/dsputil_mmx.c:2670: warning:
suggest parentheses around arithmetic in operand of |
/Users/takis/src/ffmpeg-rw/libavcodec/i386/dsputil_mmx.c:2670: warning:
suggest parentheses around arithmetic in operand of |
/Users/takis/src/ffmpeg-rw/libavcodec/i386/dsputil_mmx.c: In function
'dsputil_init_mmx':
/Users/takis/src/ffmpeg-rw/libavcodec/i386/dsputil_mmx.c:3759: warning:
assignment from incompatible pointer type
/Users/takis/src/ffmpeg-rw/libavcodec/i386/dsputil_mmx.c:3767: warning:
assignment from incompatible pointer type
/Users/takis/src/ffmpeg-rw/libavcodec/i386/dsputil_mmx.c: In function
'flac_compute_autocorr_sse2':
/Users/takis/src/ffmpeg-rw/libavcodec/i386/dsputil_mmx.c:3013: error:
can't find a register in class 'GENERAL_REGS' while reloading 'asm'
/Users/takis/src/ffmpeg-rw/libavcodec/i386/dsputil_mmx.c:3015: error:
can't find a register in class 'GENERAL_REGS' while reloading 'asm'
make[1]: *** [i386/dsputil_mmx.o] Error 1
make: *** [lib] Error 2

Takis

______________________________________________________
FFmpeg issue tracker <[EMAIL PROTECTED]>
<https://roundup.mplayerhq.hu/roundup/ffmpeg/issue294>
______________________________________________________

Reply via email to