MIOpen icon indicating copy to clipboard operation
MIOpen copied to clipboard

Dedicated primitive (e.g. `Fused`) for fused convolutions?

Open atamazov opened this issue 3 years ago • 2 comments

https://github.com/ROCmSoftwarePlatform/MIOpen/blob/4e61a3ebdfe2b07b4d331cc46832eb4a6b49941c/src/solver.cpp#L264

Can we introduce some dedicated primitive (e.g. Fused) for fused convolutions and use it in the registry? This would help to avoid hacks like this: https://github.com/ROCmSoftwarePlatform/MIOpen/pull/1212/files/db05b058f77cc838f5bbafc93fe08fd8e20c713d..f26346d12ba5f2ee472d26d7e6c207325bbe6233#r723662796

atamazov avatar Oct 06 '21 20:10 atamazov

@DrizztDoUrden Has this been resolved? Thanks!

ppanchad-amd avatar Apr 15 '24 16:04 ppanchad-amd

@ppanchad-amd No, but I can take it if we consider it valuable and urgent. Do we? It is a trivial thing to do, but merging any PR takes quite some time.

DrizztDoUrden avatar May 10 '24 10:05 DrizztDoUrden