pytorch / cpuinfo

CPU INFOrmation library (x86/x86-64/ARM/ARM64, Linux/Windows/Android/macOS/iOS)
BSD 2-Clause "Simplified" License
990 stars 310 forks source link

Enable CXX only when needed for tests #133

Closed jameshilliard closed 1 year ago

jameshilliard commented 1 year ago

We shouldn't require CXX support in the project as it's only needed for tests.

jameshilliard commented 1 year ago

Benchmarks also require C++

Enabled C++ for benchmarks as well.

Maratyszcza commented 1 year ago

CI build fails, PTAL

jameshilliard commented 1 year ago

CI build fails, PTAL

Yeah, needed to move the ENABLE_LANGUAGE block down a bit after CPUINFO_SUPPORTED_PLATFORM is defined/set.