MIOpen icon indicating copy to clipboard operation
MIOpen copied to clipboard

Use shorter name for iGEMM kernel wrapper

Open asroy opened this issue 5 years ago • 1 comments

@asroy I mean this gridwise_convolution_backward_data_implicit_gemm is always the same (except "backward_data" of course), so I would replace it with smth like gw_conv_fwd/bwd/wrw_igemm or even conv_igemm_gw_fwd/bwd/wrw...

Originally posted by @atamazov in https://github.com/ROCmSoftwarePlatform/MIOpen/pull/145

asroy avatar Apr 28 '20 13:04 asroy

@asroy @zjing14 @TejashShah Is this resolved in latest ROCm 6.1.1? Thanks!

ppanchad-amd avatar Jun 03 '24 20:06 ppanchad-amd

Thank you for bringing up the issue! However, after discussing it with the component team, we have decided to keep the current implementation. Please feel free to ask follow-up questions should further assistance be required. Thanks!

taylding-amd avatar Oct 31 '24 15:10 taylding-amd