Re: [PATCH] BZ #14059 - HAS_FMA4 check needs to also check for AVX

[Jeff Law <law at redhat dot com>]

On 05/14/2012 02:12 AM, Andreas Jaeger wrote:
> On Monday, May 14, 2012 09:52:17 Andreas Jaeger wrote:
> > Btw. here's a small test program to show that the values are setup
> > correctly.
> >
> > Currently it fails on my system (/proc/cpuinfo contains avx but
> > glibc test reports it does not).
> >
> > Hope it's usefull,
>
> Yes, was helpfull for me;)
>
> I found the problem, the assignments to set the bits are wrong.
>
> Instead of
> __cpu_features.feature[index_AVX_Usable] = 1;
>
> use
> __cpu_features.feature[index_AVX_Usable]
> 	    |= bit_AVX_Usable;
>
> The same for the FMA4 test.
FWIW, twiddled the Fedora patch to fix this bug...  Everything is still 
happy.

Jeff