search

gnuradio / volk

The Vector Optimized Library of Kernels
http://libvolk.org
GNU Lesser General Public License v3.0
542 stars 202 forks source link

Add i386 to CI #717

Open argilo opened 10 months ago

argilo commented 10 months ago

Running VOLK's test suite on i386 revealed a broken protokernel and many broken tests:

Given this success, it would seem useful to add i386 to CI to catch more future bugs.

jdemel commented 9 months ago

Looking at the fixes, I agree.

jdemel commented 9 months ago

I'm looking for a way to run i386. A native GH Action would be preferable because they are much faster. I do have some hope that MacOS13 arm64 images do that for us on MacOS.

For non-x86, we use run-on-arch. This action doesn't provide an i386 image.

argilo commented 9 months ago

GNU Radio uses a 32-bit container: https://github.com/gnuradio/gnuradio/blob/bdaf5df2bd26325864d66bfb04caf8c8617296d1/.github/workflows/make-test.yml#L100-L102

argilo commented 9 months ago

Another way of catching floating-point bugs might be to use a tool like https://github.com/verificarlo/verificarlo.

But an i386 build still would have been needed to catch #714.