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

[Jules Bergmann <jules@xxxxxxxxxxxxxxxx>]

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@xxxxxxxxxxxxxxxx
(650) 331-3385 x705