mlpack / ensmallen

A header-only C++ library for numerical optimization --
http://ensmallen.org
Other
740 stars 119 forks source link

update to C++14 standard #400

Closed conradsnicta closed 3 months ago

conradsnicta commented 3 months ago

Update C++ standard to C++14.

mlpack 4.4 already requires C++17, and Armadillo 14.0 will use C++14.

The only OS that required C++11 is RHEL 7 / CentOS 7, which is EOL as of 30 June 2024:

Using "two most recent releases of long term Linux distros" as the yardstick for compiler support. The Linux distros are: RHEL (+ clones like Rocky Linux and Alma Linux), Ubuntu LTS, and Debian.

The current two supported releases of RHEL (8 and 9), Ubuntu LTS (22.04 and 24.04), Debian (11 and 12) all provide GCC versions that support at least C++14.

RHEL 8 does not support C++17 properly, so not bumping the C++ standard to C++17.

rcurtin commented 3 months ago

Awesome, I actually did not know about [[deprecated]], nice cleanup. I'll do the same for mlpack and we can get rid of mlpack_deprecated :smile: