[flac-dev] [PATCH] for configure.ac

lvqcl lvqcl.mail at gmail.com
Mon Oct 13 09:08:14 PDT 2014


lvqcl wrote:

> 1) in config.h FLAC__HAS_X86INTRIN macro is always defined and empty,
> even if x86intrin.h is not available.
>
> 2) sse_os is defined as 'yes' or 'no', but AM_CONDITIONAL tests it for 'true':

The patch is attached. Please check it.



> 3) configure --disable-sse  prints:
> "    SSE optimizations : ................... no"
> but this option actually doesn't disable SSE, so this message
> is a bit misleading.

Instead of the current message

	echo "    SSE optimizations : ................... ${sse_os}"

it's technically more correct to write:

	echo "    OS is guaranteed to support SSE : ..... ${sse_os}"

but this message is much less comprehensible.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: conf_ac.patch
Type: application/octet-stream
Size: 1074 bytes
Desc: not available
Url : http://lists.xiph.org/pipermail/flac-dev/attachments/20141013/8e7d3f18/attachment.obj 


More information about the flac-dev mailing list