Skip to content

Commit d0c3504

Browse files
authored
Merge pull request #3631 from martin-frbg/revertdynskx
Revert selection of a different DGEMM kernel for SkylakeX in DYNAMIC_ARCH builds
2 parents faf58d2 + dac14a5 commit d0c3504

File tree

2 files changed

+1
-26
lines changed

2 files changed

+1
-26
lines changed

kernel/x86_64/KERNEL.SKYLAKEX

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -20,19 +20,14 @@ SGEMM_SMALL_K_B0_TN = sgemm_small_kernel_tn_skylakex.c
2020
SGEMM_SMALL_K_TT = sgemm_small_kernel_tt_skylakex.c
2121
SGEMM_SMALL_K_B0_TT = sgemm_small_kernel_tt_skylakex.c
2222

23-
ifndef DYNAMIC_ARCH
2423
DGEMMKERNEL = dgemm_kernel_16x2_skylakex.c
2524
DTRMMKERNEL = dgemm_kernel_16x2_skylakex.c
2625
DGEMMINCOPY = ../generic/gemm_ncopy_16.c
2726
DGEMMITCOPY = dgemm_tcopy_16_skylakex.c
2827
DGEMMONCOPY = ../generic/gemm_ncopy_2.c
2928
DGEMMOTCOPY = ../generic/gemm_tcopy_2.c
3029
DTRSMKERNEL_RN = ../generic/trsm_kernel_RN.c
31-
else
32-
DGEMMKERNEL = dgemm_kernel_4x8_skylakex_2.c
33-
DGEMMONCOPY = dgemm_ncopy_8_skylakex.c
34-
DGEMMOTCOPY = dgemm_tcopy_8_skylakex.c
35-
endif
30+
3631
DGEMM_SMALL_M_PERMIT = dgemm_small_kernel_permit_skylakex.c
3732
DGEMM_SMALL_K_NN = dgemm_small_kernel_nn_skylakex.c
3833
DGEMM_SMALL_K_B0_NN = dgemm_small_kernel_nn_skylakex.c

param.h

Lines changed: 0 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -1669,22 +1669,14 @@ USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
16691669
#else
16701670

16711671
#define SGEMM_DEFAULT_UNROLL_M 16
1672-
#ifdef DYNAMIC_ARCH
1673-
#define DGEMM_DEFAULT_UNROLL_M 4
1674-
#else
16751672
#define DGEMM_DEFAULT_UNROLL_M 16
1676-
#endif
16771673
#define QGEMM_DEFAULT_UNROLL_M 2
16781674
#define CGEMM_DEFAULT_UNROLL_M 8
16791675
#define ZGEMM_DEFAULT_UNROLL_M 4
16801676
#define XGEMM_DEFAULT_UNROLL_M 1
16811677

16821678
#define SGEMM_DEFAULT_UNROLL_N 4
1683-
#ifdef DYNAMIC_ARCH
1684-
#define DGEMM_DEFAULT_UNROLL_N 8
1685-
#else
16861679
#define DGEMM_DEFAULT_UNROLL_N 2
1687-
#endif
16881680
#define QGEMM_DEFAULT_UNROLL_N 2
16891681
#define CGEMM_DEFAULT_UNROLL_N 2
16901682
#define ZGEMM_DEFAULT_UNROLL_N 2
@@ -1718,29 +1710,17 @@ USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
17181710
#else
17191711

17201712
#define SGEMM_DEFAULT_P 448
1721-
#ifndef DYNAMIC_ARCH
17221713
#define DGEMM_DEFAULT_P 192
1723-
#else
1724-
#define DGEMM_DEFAULT_P 384
1725-
#endif
17261714
#define CGEMM_DEFAULT_P 384
17271715
#define ZGEMM_DEFAULT_P 256
17281716

17291717
#define SGEMM_DEFAULT_Q 448
1730-
#ifndef DYNAMIC_ARCH
17311718
#define DGEMM_DEFAULT_Q 384
1732-
#else
1733-
#define DGEMM_DEFAULT_Q 168
1734-
#endif
17351719
#define CGEMM_DEFAULT_Q 192
17361720
#define ZGEMM_DEFAULT_Q 128
17371721

17381722
#define SGEMM_DEFAULT_R sgemm_r
1739-
#ifndef DYNAMIC_ARCH
17401723
#define DGEMM_DEFAULT_R 8640
1741-
#else
1742-
#define DGEMM_DEFAULT_R 13824
1743-
#endif
17441724
#define CGEMM_DEFAULT_R cgemm_r
17451725
#define ZGEMM_DEFAULT_R zgemm_r
17461726

0 commit comments

Comments
 (0)