[flac-dev] Description of disable-sse option
lvqcl.mail
lvqcl.mail at gmail.com
Sun Dec 4 11:37:35 UTC 2016
OS SSE support is now mandatory, so the description of --disable-sse
option isn't correct now (actually, it wasn't fully correct even before
this).
*Currently all it does is disables -msse2 flag*
The relevant parts of configure.ac:
AC_ARG_ENABLE(sse,
AC_HELP_STRING([--disable-sse], [Disable SSE if the OS does not support
SSE instructions]),
[case "${enableval}" in
yes) sse_os=yes ;;
no) sse_os=no ;;
*) AC_MSG_ERROR(bad value ${enableval} for --enable-sse) ;;
esac],[sse_os=yes])
if test "x$asm_optimisation$sse_os" = "xyesyes" ; then
XIPH_ADD_CFLAGS([-msse2])
fi
So maybe it's better to rename it to --disable-sse2 or
--disable-forced-sse2
or --no-force-sse2? (and rename 'sse_os' to 'use_sse2' or 'forced-sse2' ?)
Or least the description should be updated so that it reflects
the current function of this option.
More information about the flac-dev
mailing list