[Mesa-dev] [PATCH] swr: fix -march flag for AVX

[Ilia Mirkin]

On Fri, Jun 10, 2016 at 3:43 PM, Tim Rowley <timothy.o.rowley at intel.com> wrote:
> Previously used core-avx-i was for ivybridge;
> corei7-avx allows sandybridge.
> ---
>  src/gallium/drivers/swr/Makefile.am | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/gallium/drivers/swr/Makefile.am b/src/gallium/drivers/swr/Makefile.am
> index d211f2e..8156cf2 100644
> --- a/src/gallium/drivers/swr/Makefile.am
> +++ b/src/gallium/drivers/swr/Makefile.am
> @@ -124,7 +124,7 @@ COMMON_LDFLAGS = \
>  lib_LTLIBRARIES = libswrAVX.la libswrAVX2.la
>
>  libswrAVX_la_CXXFLAGS = \
> -       -march=core-avx-i \
> +       -march=corei7-avx \

Just wondering if it'd be enough to say like

-march=x86_64
-mavx

and add -mavx2 for libswrAVX2.

I suspect you've iterated through this 20 times and this has some
shortcoming I'm not thinking of, but figured I'd point it out just in
case it helps.

  -ilia

>         -DKNOB_ARCH=KNOB_ARCH_AVX \
>         $(COMMON_CXXFLAGS)
>
> --
> 1.9.1
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev