Check that number of convolution groups is multiple of merged groups.

This commit is contained in:
Ville Pietilä
2025-10-07 15:05:09 +00:00
parent 438787dbb6
commit fdfbd1e770

View File

@@ -626,17 +626,17 @@ struct GroupedConvolutionBackwardWeightKernel
return false;
}
// if constexpr (GroupedConvTraitsType_::NumGroupsToMerge > 1)
// {
// const index_t ConvG = kargs.wei_g_k_c_xs_lengths[number<0>{}];
// if (ConvG % GroupedConvTraitsType_::NumGroupsToMerge != 0)
// {
// CK_TILE_ERROR("ConvG must be a multiple of NumGroupsToMerge!");
// return false;
// }
if constexpr(GroupedConvTraitsType_::NumGroupsToMerge > 1)
{
const index_t ConvG = kargs.wei_g_k_c_xs_lengths[number<0>{}];
if(ConvG % GroupedConvTraitsType_::NumGroupsToMerge != 0)
{
CK_TILE_ERROR("ConvG must be a multiple of NumGroupsToMerge!");
return false;
}
// // TODO: Should we also check that GemmM <= MPerBlock and GemmN <= NPerBlock?
// }
// TODO: Should we also check that GemmM <= MPerBlock and GemmN <= NPerBlock?
}
return true;
}