Skip to content

Commit 28c54e7

Browse files
Fixed F32 cntx override in DE. (#493)
This PR adjusts gemmF32DEBackend’s GEMV fast-path configuration in the decision engine to better reflect the intended KC source and to avoid unnecessarily overriding KC for AVX512 configurations. AMD-Internal: [SWLCSG-4146] Signed-off-by: Mithun Mohan <MithunMohan.KadavilMadanaMohanan@amd.com>
1 parent 6343b43 commit 28c54e7

1 file changed

Lines changed: 1 addition & 2 deletions

File tree

src/include/decision_engine/de_backend.hh

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -233,7 +233,7 @@ class gemmF32DEBackend : public iDEBackend
233233
? 16
234234
: 64;
235235
k_unroll = 2;
236-
kc = 512; // This is hardcoded from ZEN4 context.
236+
kc = 512; // This is hardcoded from ZEN3 context.
237237
} else if (isAvx512) {
238238
mr = 1;
239239
nr = 64;
@@ -242,7 +242,6 @@ class gemmF32DEBackend : public iDEBackend
242242
== kernel_frame::kernelInstrPreference::avx512_ymm_favour)
243243
? 2
244244
: 4;
245-
kc = 512;
246245
} else {
247246
return INVALID_KERNEL_INFO;
248247
}

0 commit comments

Comments
 (0)