baidu-research / DeepBench

Benchmarking Deep Learning operations on different hardware
Apache License 2.0
1.07k stars 239 forks source link

Fix the parameter order of time_cnn in ARM convolution bench #124

Closed jaidTw closed 1 month ago

jaidTw commented 3 years ago

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);
hhl-art commented 1 month ago

这是来自QQ邮箱的假期自动回复邮件。你好,我最近正在休假中,无法亲自回复你的邮件。我将在假期结束后,尽快给你回复。