On Thu, Mar 31, 2011 at 2:44 PM, Marcus D. Leech <mle...@ripnet.com> 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
_______________________________________________ Discuss-gnuradio mailing list Discuss-gnuradio@gnu.org http://lists.gnu.org/mailman/listinfo/discuss-gnuradio
