[FFmpeg-cvslog] r16488 - in trunk: configure libavutil/mem.c

ramiro ramiro
Sun Jan 11 03:16:31 CET 2009


Michael Niedermayer wrote:
> On Fri, Jan 09, 2009 at 03:35:52AM -0200, ramiro wrote:
[...]
>> So, what could be done here?
>> - check gcc version in configure and have sse depend on it
>> or
>> - #if defined(HAVE_SSE) && (!defined(__GNUC__) || __GNUC__ > 2)
>> or
>> - disable HAVE_SSE in some internal header based on __GNUC__
>>
>> I'm slightly more in favor of configure, because then we can keep 
>> HAVE_SSE as the only basis to disable building SSE code in the future. 
>> This patch only disabled selecting it at run-time. The functions are 
>> still being built.
> 
> fine though you need approval from the configure maintainers for this

Patch attached.

Ramiro Polla
-------------- next part --------------
A non-text attachment was scrubbed...
Name: sse_gcc_2_nono.diff
Type: text/x-patch
Size: 607 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-cvslog/attachments/20090111/a924514a/attachment.bin>



More information about the ffmpeg-cvslog mailing list