Johannes Demel
Johannes Demel
I totally agree. We need to sweep over a range of vector inputs. This should be the default. With vector size `
In order to add this to our test system, we need to put a lot of effort into: https://github.com/gnuradio/volk/blob/c9d42a4bf31af53ec2d81bbd1dec8754feb7303f/lib/qa_utils.cc#L511 And the whole infrastructure there. Also, this refactoring/rewrite would be required...
We have 3 kernels here: generix, avx2 and spiral. Avx2/spiral both yield a ~6x speed up and VOLK tests pass. We don't want to break API, thus we can't remove...
By now, I could reproduce the error. One of the AVX2 kernels is broken. Unfortunately, the bug only manifests itself in this GR example flowgraph. I couldn't make it fail...
From [gnuradio#3376](https://github.com/gnuradio/gnuradio/issues/3376): There's a workaround for volk_8u_x4_conv_k7_r2_8u`. Run `volk_profile`, then inspect `~/.volk/volk_config`. Find the line that starts with: ``` volk_8u_x4_conv_k7_r2_8u ``` and make sure, it ends with spiral spiral (or...
@armin485 may I ask you to do the following things: - Open a new issue for the polar encoder and decoder kernels - Describe how you make these fail -...
I agree. @michaelld if you agree too, I suggest we close this issue.
https://github.com/gnuradio/volk/blob/af69c60e908cd64b3bacd4ee6b4ce4a52b711865/tmpl/volk_cpu.tmpl.c#L101-L108 I had a closer look at your second comment. I assume you refer to the `volk_get_`lvarch` function. It is only used in `volk.c`, e.g. https://github.com/gnuradio/volk/blob/af69c60e908cd64b3bacd4ee6b4ce4a52b711865/tmpl/volk.tmpl.c#L48 It is related to...
So your goal is to start a new volk module? This does require some infrastructure from VOLK but it would be really confusing to carry around the whole git history....
Would you mind to open a PR to fix this issue?