Maratyszcza / NNPACK

Acceleration package for neural networks on multi-core CPUs
BSD 2-Clause "Simplified" License
1.68k stars 316 forks source link

Feature request: broader winograd 3x3 convolution stride 2 support #137

Closed ghost closed 6 years ago

ghost commented 6 years ago

Hi,

I noticed in the sources winograd 3x3 stride 2 convolution support for neon cpu. As this is a very widely used layer are there plans to make this also work on non neon cpu's (intel/amd)? Would be a very useful addition and great speed-up.

Thanks

Maratyszcza commented 6 years ago

3x3 stride 2 Winograd for NEON wasn't much of a win, so no plans to port it to other platforms.

lironmo commented 6 years ago

@Maratyszcza, Thank you very much for taking the time to answer. We are planning to develop strided convolution for training, in order to support training.

we wondered why "3x3 stride 2 Winograd for NEON wasn't much of a win", compare to what ?

victorygogogo commented 5 years ago

@lironmo hello how about your stride2 winograd convolution ?can you share some infomation about it?