Skip to content

Commit

Permalink
added miopenGetConvolutionSpatialDim if ROCm5.5 (#14772)
Browse files Browse the repository at this point in the history
The API should be `miopenGetConvolutionSpatialDim(cdesc, &spatial_dim)`, NOT `miopenGetConvolutionDescriptorSize(cdesc, &spatial_dim)`
  • Loading branch information
zstreet87 authored Mar 3, 2023
1 parent 0ebe8e3 commit 6e2ca15
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion onnxruntime/contrib_ops/rocm/fused_conv.cc
Original file line number Diff line number Diff line change
Expand Up @@ -75,7 +75,7 @@ struct FNVHash {
void HashConvolutionDescriptor(miopenConvolutionDescriptor_t cdesc) {
int spatial_dim = 1;
#if ROCM_VERSION >= 50500
miopenGetConvolutionDescriptorSize(cdesc, &spatial_dim);
miopenGetConvolutionSpatialDim(cdesc, &spatial_dim);
#else
// Previous versions of MIOpen doesn't provide API to probe the dimension of a
// miopenConvolutionDescriptor_t, so we have to guess.
Expand Down

0 comments on commit 6e2ca15

Please sign in to comment.