cpu: x64: gemm: correct status propagation

oneapi-src · Aug 19, 2021 · 45e3039 · 45e3039
1 parent fd6d14c
commit 45e3039
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 4 deletions.
diff --git a/src/cpu/x64/gemm/f32/jit_avx512_common_gemm_f32.cpp b/src/cpu/x64/gemm/f32/jit_avx512_common_gemm_f32.cpp
@@ -1743,7 +1743,7 @@ xbyak_gemm_t *get_xbyak_gemm(
     // Kernel table [isTransA][isTransB][hasBias][beta (0, 1, other)]
     static std::unique_ptr<xbyak_gemm_t> kernel_table[2][2][2][3];
     static std::once_flag initialized;
-    dnnl_status_t st = dnnl_success;
+    static dnnl_status_t st = dnnl_success;
     std::call_once(initialized, [&] {
         for (bool isTransA : {false, true})
             for (bool isTransB : {false, true})

diff --git a/src/cpu/x64/gemm/f32/jit_avx512_core_gemm_smalln_tn_f32_kern.cpp b/src/cpu/x64/gemm/f32/jit_avx512_core_gemm_smalln_tn_f32_kern.cpp
@@ -727,7 +727,7 @@ dnnl_status_t sgemm_smalln_tn(const dim_t m, const dim_t n, const dim_t k,
     static std::unique_ptr<xbyak_gemm_smalln_tn_t> kernels[4][3][3];
     static std::once_flag initialized;
 
-    dnnl_status_t st = dnnl_success;
+    static dnnl_status_t st = dnnl_success;
     std::call_once(initialized, [&] {
         for (dim_t N : {1, 2, 3, 4}) {
             for (float al : {0.0f, 1.0f, 2.0f}) {

diff --git a/src/cpu/x64/gemm/f32/jit_avx_gemm_f32.cpp b/src/cpu/x64/gemm/f32/jit_avx_gemm_f32.cpp
@@ -2196,7 +2196,7 @@ xbyak_gemm_t *get_xbyak_gemm(
     // Kernel table [isTransA][isTransB][hasBias][beta (0, 1, other)]
     static std::unique_ptr<xbyak_gemm_t> kernel_table[2][2][2][3];
     static std::once_flag initialized;
-    dnnl_status_t st = dnnl_success;
+    static dnnl_status_t st = dnnl_success;
     std::call_once(initialized, [&] {
         for (bool isTransA : {false, true})
             for (bool isTransB : {false, true})

diff --git a/src/cpu/x64/gemm/gemm_info.cpp b/src/cpu/x64/gemm/gemm_info.cpp
@@ -354,7 +354,7 @@ void gemm_info_t<a_t, b_t, c_t>::jit_init(void) {
     const int um = this->um;
 
     static std::once_flag initialized;
-    dnnl_status_t st = dnnl_success;
+    static dnnl_status_t st = dnnl_success;
     std::call_once(initialized, [&, um] {
         const bool b_is_s8 = data_traits<b_t>::data_type == data_type::s8;
         constexpr bool is_int8 = utils::one_of(