[Mesa-dev] [PATCH 2/2] configure/swr: add KNL and SKX architecture targets

[Emil Velikov]

On 17 July 2017 at 15:08, Tim Rowley <timothy.o.rowley at intel.com> wrote:
> Not built by default.
> ---
>  configure.ac                           | 16 ++++++++++++++
>  src/gallium/drivers/swr/Makefile.am    | 38 ++++++++++++++++++++++++++++++++++
>  src/gallium/drivers/swr/swr_loader.cpp | 20 +++++++++++++++++-
>  3 files changed, 73 insertions(+), 1 deletion(-)
>
> diff --git a/configure.ac b/configure.ac
> index 3a8fa4d7ea..4437c8189d 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -2518,6 +2518,20 @@ if test -n "$with_gallium_drivers"; then
>                      AC_SUBST([SWR_AVX2_CXXFLAGS])
>                      HAVE_SWR_AVX2=yes
>                      ;;
> +                xknl)
> +                    swr_require_cxx_feature_flags "KNL" "defined(__AVX512F__) && defined(__AVX512ER__)" \
> +                        ",-march=knl,-xMIC-AVX512" \
> +                        SWR_KNL_CXXFLAGS
> +                    AC_SUBST([SWR_KNL_CXXFLAGS])
> +                    HAVE_SWR_KNL=yes
> +                    ;;
> +                xskx)
> +                    swr_require_cxx_feature_flags "SKX" "defined(__AVX512F__) && defined(__AVX512BW__)" \
> +                        ",-march=skylake-avx512,-xCORE-AVX512" \
> +                        SWR_SKX_CXXFLAGS
> +                    AC_SUBST([SWR_SKX_CXXFLAGS])
> +                    HAVE_SWR_SKX=yes
> +                    ;;
Please update of the help string. Otherwise these two are completely
undocumented.

Can I bribe you to add a Travis entries for the above? If it doesn't
take too long to build, you can it squash into the existing ones.

Thanks
Emil