Skip to content

Commit c5474a6

Browse files
committed
added assert for aborting and fixed comment
1 parent 5abfd16 commit c5474a6

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

ggml/src/ggml-vulkan/ggml-vulkan.cpp

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -5105,13 +5105,13 @@ static vk_matmul_pipeline ggml_vk_get_mul_mat_mat_id_pipeline(ggml_backend_vk_co
51055105
} else if (ctx->device->coopmat_acc_f32_support) {
51065106
return mmp.f32acc;
51075107
} else {
5108-
// No supported pipeline
5109-
return nullptr;
5108+
// f16acc nor f32acc is supported so we just abort
5109+
GGML_ASSERT(false);
51105110
}
51115111
}
5112-
// coopmat2 or no coopmat.
5112+
// coopmat2 or no coopmat comes here.
51135113
// coopmat2 should support both f16acc and f32acc by default with all child pipelines.
5114-
// No coopmat should support both f16acc and f32acc when FP16 is preferred, but only f32acc when not preferred.
5114+
// No coopmat should support both f16acc and f32acc when FP16 is preferred, but only f32acc when FP16 is not preferred.
51155115
return prefer_fp16acc ? mmp.f16acc : mmp.f32acc;
51165116
}
51175117

0 commit comments

Comments
 (0)