[vsipl++] [patch] Fix SSE2 mag()

[Jules Bergmann]

Jules Bergmann wrote:
> Stefan Seefeld wrote:
>> Jules Bergmann wrote:
>>
>>> Not sure why this wasn't showing up with buildbot.  Perhaps the 
>>> buildbot configs are not using --enable-simd-loop-fusion?  Also, this 
>>> wouldn't have shown up when IPP was enabled because IPP mag takes 
>>> precendence of SIMD loop fusion.
>>
>> That's strange indeed. As demonstrated here:
>>
>> https://intranet.codesourcery.com/buildbot/vsiplxx/builders/SerialBuiltin32/builds/8/steps/build_bdist/logs/stdio 
>>
>>
>> buildbot does use --enable-simd-loop-fusion, and no IPP is present in 
>> the 'Builtin' build variants. I'm not sure why the appropriate test 
>> didn't show the failure.
> 
> Hmm, sure does.
> 
> That's a 32-bit build (-m32).  Does buildbot cover SerialBuiltin64?
> 
> I wouldn't expect SSE2 to be disabled in 32-bit, but ...
> 

Interesting.

SerialBuiltin32-release (is that the same as SerialBuiltin32 builder?) 
shows the following for check_config.cpp:

Sourcery VSIPL++ Library Configuration
   VSIP_IMPL_PAR_SERVICE           - 0 (Serial)
   VSIP_IMPL_IPP                   - 0
   VSIP_IMPL_SAL                   - 0
   VSIP_IMPL_CBE_SDK               - 0
   VSIP_IMPL_HAVE_SIMD_LOOP_FUSION - 0
   VSIP_IMPL_HAVE_SIMD_GENERIC     - 0
   VSIP_IMPL_PREFER_SPLIT_COMPLEX  - 0
   VSIP_IMPL_HAS_EXCEPTIONS        - 0
   VSIP_IMPL_ALLOC_ALIGNMENT       - 32
   __SSE__                         - 0
   __SSE2__                        - 0
   __VEC__                         - 0
   _MC_EXEC                        - 0
Sourcery VSIPL++ Compiler Configuration
   __GNUC__                        - 3


VSIP_IMPL_HAVE_SIMD_LOOP_FUSION is 0.  So is __SSE__ and __SSE2__.

see 
https://intranet.codesourcery.com/buildbot/vsiplxx/report/SerialBuiltin32-release.html#check_config.cpp

-- 
Jules Bergmann
CodeSourcery
jules at codesourcery.com
(650) 331-3385 x705