On Jul 17, 2017, at 11:51 AM, Emil Velikov <emil.l.veli...@gmail.com<mailto:emil.l.veli...@gmail.com>> wrote:
On 17 July 2017 at 15:08, Tim Rowley <timothy.o.row...@intel.com<mailto:timothy.o.row...@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. Will do. 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. I’ll do that in a future patch; the avx512 code in swr as it currently exists in mesa-master only works on icc, which isn’t one of the Travis options. We’re working on a patch which enables modern clang and gcc to also build it. Thanks Emil
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev