ARM compiler options are wrong

Open knell opened this issue 4 years ago • 3 comments

16:37:39 [ 8%] Building C object PFFFT/CMakeFiles/PFFFT.dir/pffft_double.c.o 16:37:39 arm-buildroot-linux-gnueabihf-gcc.br_real: error: unrecognized command-line option '-msse2'

Starting from line 170 in CMakeLists.txt there is no check for arm platforms, like this:

elseif(CMAKE_COMPILER_IS_GNUCC AND NOT USE_SIMD_NEON)

Dec 13 '21 16:12 knell

Hi @knell, thank you for the bug report. I don't have time to take care of it, but I'm happy to accept pull requests.

Dec 14 '21 08:12 marton78

Well, the problem is that the -march=native does not work as well, it's not a valid option too. I have fixed it using the particular armv7 architecture but that's not the proper fix in general, of course. It should be some precise platform detection I guess to use that option properly.

Dec 15 '21 15:12 knell

would you please check this again? (after my big changes in the cmake files)

Apr 08 '22 19:04 hayguen