Closed madmann91 closed 4 years ago
Actually that set(CMAKE_C_COMPILER "gcc-9")
is in there by accident. I just added it for testing, and indeed one should just do cmake . -DCMAKE_C_COMPILER=<compiler>
instead. The helper scripts in the rebuild
directory do just this.
And yeah, after seeing the massive performance boost from a little tweak to CMakeLists.txt
, I was a bit anxious to commit it in as quickly as possible. I'll apply that check you outlined shortly.
I think requiring CMake 3.9
and newer is fine, all my experiments with running this on ancient hardware use plain shell scripts for the build system anyway.
Resolved by d56da635ed13d96c8fc5a1d1f8e2e4870b5602d4
The most recent commits break the build on my system. I have
gcc
in version 10, so the lineset(CMAKE_C_COMPILER "gcc-9")
breaks everything for me. I believe it would be better to set that per-build, if required with the CMake GUI, instead of forcing a specific version by default. Alternatively, it is possible to use the following example to detect if inter-procedural optimization is supported by the compiler (starting from CMake 3.9.6):(reference: https://cmake.org/cmake/help/v3.9/module/CheckIPOSupported.html)