DeepBench icon indicating copy to clipboard operation
DeepBench copied to clipboard

Fix the parameter order of time_cnn in ARM convolution bench

Open jaidTw opened this issue 3 years ago • 0 comments

Hi, I was looking at the code of convolution benchmark on ARM, and found the calling order and declaration order of parameters of time_cnn does not match. I believe the order of 'c' and 'n' should be exchanged.

Declaration:

int time_cnn(unsigned int w, unsigned int h, unsigned int n, unsigned int c, 
             unsigned int k, unsigned int filter_w, int filter_h,
             unsigned int pad_w, unsigned int pad_h, 
             unsigned int wstride, unsigned int hstride,

Calling:

auto time = time_cnn(w, h, c, n, k, filter_w, filter_h, pad_w, pad_h, wstride, hstride, num_repeats);

jaidTw avatar Apr 25 '21 09:04 jaidTw