amd/blis
1
0
Fork 0
mirror of https://github.com/amd/blis.git synced 2026-04-20 07:38:53 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix: Resolve Operator Precedence Warning in Zen5 DCOMPLEX Threshold Logic

Browse Source 
- Add explicit parentheses around (n <= 1520) && (k <= 128) to clarify
  operator precedence and resolve compiler warning. The intended logic
  is (m <= 1380) OR (n <= 1520 AND k <= 128).

- This change eliminates the compiler warning about mixing || and &&
  operators without explicit grouping.
This commit is contained in:
S, Hari Govind
2025-10-24 14:23:03 +05:30
committed by GitHub
parent 1e2a8db100
commit 64d8d06aad
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
kernels/zen5/aocl_smart/bli_aocl_smart_zen5.c
View File

@@ -87,7 +87,7 @@ bool bli_cntx_gemmsup_thresh_is_met_zen5( obj_t* a, obj_t* b, obj_t* c, cntx_t*
// The threshold for m is a single value, but for n, it is
// also based on the packing size of A, since the kernels are
// column preferential
if ( ( ( m <= 1380 ) || ( n <= 1520 ) && ( k <= 128 ) ) && ( m + n + k < 6400 ) ) return TRUE;
if ( ( ( m <= 1380 ) || ( ( n <= 1520 ) && ( k <= 128 ) ) ) && ( m + n + k < 6400 ) ) return TRUE;
return FALSE;
}