Unverified Commit 236bd148 authored by Anthony Chang's avatar Anthony Chang Committed by GitHub
Browse files

Fix split-k gemm test (#231)



* properly return error flag; reveals bug in split-k gemm

* fix bug in split k

* update split-k test case
Co-authored-by: default avatarChao Liu <chao.liu2@amd.com>
parent 0e9c88ce
...@@ -226,9 +226,8 @@ int main(int argc, char* argv[]) ...@@ -226,9 +226,8 @@ int main(int argc, char* argv[])
std::vector<gemmArgs> test_cases; std::vector<gemmArgs> test_cases;
if(argc == 1) if(argc == 1)
{ {
test_cases = {{GemmMatrixLayout::MK_KN_MN, 3, 3, 3, 3, 3, 3, 1}}; test_cases = {{GemmMatrixLayout::MK_KN_MN, 1024, 1024, 1024, 1024, 1024, 1024, 2},
// JD: Populate with more and meaningful {GemmMatrixLayout::MK_KN_MN, 1024, 1024, 1024, 1024, 1024, 1024, 8}};
return 0;
} }
else if(argc == 9) else if(argc == 9)
{ {
...@@ -253,11 +252,10 @@ int main(int argc, char* argv[]) ...@@ -253,11 +252,10 @@ int main(int argc, char* argv[])
printf("arg2 to 7: M, N, K, StrideA, StrideB, StrideC KBatch\n"); printf("arg2 to 7: M, N, K, StrideA, StrideB, StrideC KBatch\n");
return -1; return -1;
} }
bool error = false;
for(const auto& kinder : test_cases) for(const auto& kinder : test_cases)
{ {
const auto res = test_gemm(kinder); error |= test_gemm(kinder);
if(!res)
return -1;
} }
return 0; return error ? 1 : 0;
} }
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment