[flac-dev] [PATCH] configure: only use -mstackrealign for mingw32

[Dave Yeo]

On 04/10/15 02:10 pm, Erik de Castro Lopo wrote:
> Tristan Matthews wrote:
>
>> ---
>>   configure.ac | 4 +++-
>>   1 file changed, 3 insertions(+), 1 deletion(-)
>>
>> diff --git a/configure.ac b/configure.ac
>> index eb9b0cc..e7d68c3 100644
>> --- a/configure.ac
>> +++ b/configure.ac
>> @@ -399,9 +399,11 @@ if test x$ac_cv_c_compiler_gnu = xyes ; then
>>
>>   	if test "x$asm_optimisation$sse_os" = "xyesyes" ; then
>>   		XIPH_ADD_CFLAGS([-msse2])
>> -		XIPH_ADD_CFLAGS([-mstackrealign])
>>   		fi
>> +	fi
>>
>> +if test "$host_os" = "mingw32" ; then
>> +	XIPH_ADD_CFLAGS([-mstackrealign])
>>   	fi
>>
>>   XIPH_ADD_CFLAGS([-Wextra])
>
> Everyone happy with this patch?
>

Should it be a case statement so other OSes can be added or just use || 
? OS/2 should also have the -mstackrealign option added as well.
Dave