Re: [Discuss-gnuradio] Build error on Ubuntu 10.04 on x86

[Tom Rondeau]

On Thu, Mar 31, 2011 at 2:44 PM, Marcus D. Leech <address@hidden> wrote:

On 03/31/2011 01:36 PM, Nick Foster wrote:

On Wed, 2011-03-30 at 20:44 -0400, Marcus D. Leech wrote:

That's it, all right. volk_common.h defines a union which includes
__m128i, which is only defined in SSE2 (emmintrin.h) and higher -- but
the test Volk uses for this union is only (if LV_HAS_MMX). That test
should be (if LV_HAS_SSE2). Ordinarily, on SSE2 machines, there's an
#ifdef at the end of xmmintrin.h which includes emmintrin, where __m128i
is defined.

I don't know what making that change will break on non-SSE2 machines;
can you try it?

--n

Funortunately, it's not my machine.  I was asking on behalf of a customer, who has decided to go off and
 purchase a more-modern machine, which likely has "all the microcode bells and whistles" :-)

But we do need to deal with this for older, non-SSE2-or-better machines.

I agree, and this looks like a mistake. Wish we had the machine to test it on :)

Tom