flac dev - Mar 2014 - PATCH: OS SSE support detection, version 2

The second version of the patch for OS SSE support detection

part 1: fixes

part 2: new code

Any comments?
-------------- next part --------------
A non-text attachment was scrubbed...
Name: cpu_part1.patch
Type: application/octet-stream
Size: 3103 bytes
Desc: not available
Url :
http://lists.xiph.org/pipermail/flac-dev/attachments/20140311/62b5ba0c/attachment.obj
-------------- next part --------------
A non-text attachment was scrubbed...
Name: cpu_part2.patch
Type: application/octet-stream
Size: 3654 bytes
Desc: not available
Url :
http://lists.xiph.org/pipermail/flac-dev/attachments/20140311/62b5ba0c/attachment-0001.obj

lvqcl wrote:
> The second version of the patch for OS SSE support detection
> 
> part 1: fixes
> 
> part 2: new code
> 
> Any comments?
Applied cpu_part2.patch. Thanks!

Erik
-- 
----------------------------------------------------------------------
Erik de Castro Lopo
http://www.mega-nerd.com/

Erik de Castro Lopo wrote:
>> part 1: fixes
>>
>> part 2: new code
>>
>> Any comments?
>
> Applied cpu_part2.patch. Thanks!
Thanks. I hope that this code will help to avoid "bug" reports
such as http://sourceforge.net/p/flac/bugs/409/ (it seems that
the author of this report compiled FLAC without --enable-sse option).

OTOH, SSE support is unavailable only in Win95 and WinNT. Also,
as Martijn van Beurden pointed out, flac.exe doesn't support Win95
(and also 98/Me) since version 1.3.0, and nobody complained about it.

Maybe it's better and simpler to change configure.ac so that --enable-sse
is on by default if the target OS is Windows?



About part 1:does it have any problems? I can split it into several patches
(1 fix = 1 patch) or explain the changes in it in detail.