[patch] Move the popcnt intrinsics to a separate file

Richard Henderson rth@redhat.com
Fri Dec 4 17:46:00 GMT 2009


On 12/04/2009 09:36 AM, Jakub Jelinek wrote:
> abmintrin.h can be only included from within x86intrin.h (otherwise it
> #errors). _mm_popcnt_u{32,64} are defined both in abmintrin.h and
> in smmintrin.h (which is included first from x86intrin.h).  When
> compiling x86intrin.h with -msse4.2 -mabm (guess no CPU currently implements
> that, but it isn't a rejected options combination either), I was getting errors
> because _mm_popcnt_u{32,64} was defined twice.

Ah.  Then perhaps we should go back to one of Sebastian's previous 
patches that provided a popcount header?  We could include that in the 
supposedly stand-alone smmintrin.h as well as from x86intrin.h and 
protected by a POPCNT isa cpp define.  Given that there's nothing else 
in the ABM header yet, we could just omit that for now.


r~



More information about the Gcc-patches mailing list