From 815c7ab15e41cfc5d7acf04552485a0593d2b33c Mon Sep 17 00:00:00 2001 From: Egbert Eich Date: Tue, 1 Mar 2022 19:27:47 +0100 Subject: [PATCH] For DYNAMIC_ARCH don't use sbgemm_r as parameter.c doesn't get build Presently, DYNAMIC_ARCH doesn't build if the build includes Intel Cooperlake or Sapphire Rapids cores. This is because their init function reference sbgemm_r which is defined in driver/other/parameter.c. This file is not built when using DYNAMIC_ARCH. The value is the one that blas_set_parameter() would set on build without DYNAMIC_ARCH. There seems to be some duplication between blas_set_parameter() and the kernel specific init_parameter() calls. Some consolidation would be in order here. Signed-off-by: Egbert Eich --- param.h | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/param.h b/param.h index 8649e4486..e8fdfaeab 100644 --- a/param.h +++ b/param.h @@ -1803,7 +1803,11 @@ USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. #define SBGEMM_DEFAULT_UNROLL_M 32 #define SBGEMM_DEFAULT_P 256 #define SBGEMM_DEFAULT_Q 1024 +#ifndef DYNAMIC_ARCH #define SBGEMM_DEFAULT_R sbgemm_r +#else +#define SBGEMM_DEFAULT_R 43280 +#endif #ifdef ARCH_X86 @@ -1933,7 +1937,11 @@ USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. #define SBGEMM_DEFAULT_UNROLL_M 16 #define SBGEMM_DEFAULT_P 384 #define SBGEMM_DEFAULT_Q 768 +#ifndef DYNAMIC_ARCH #define SBGEMM_DEFAULT_R sbgemm_r +#else +#define SBGEMM_DEFAULT_R 43280 +#endif #ifdef ARCH_X86