Skip to content

CMake cleanup#1305

Merged
serge-sans-paille merged 2 commits intoxtensor-stack:masterfrom
AntoinePrv:cmake
Apr 16, 2026
Merged

CMake cleanup#1305
serge-sans-paille merged 2 commits intoxtensor-stack:masterfrom
AntoinePrv:cmake

Conversation

@AntoinePrv
Copy link
Copy Markdown
Contributor

No description provided.

Comment thread CMakeLists.txt

cmake_minimum_required(VERSION 3.10)
project(xsimd)
option(XSIMD_REFACTORING ON)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

XSIMD_REFACTORING, that brings back some memories :-)

Comment thread test/doc/CMakeLists.txt
if(${CMAKE_SYSTEM_PROCESSOR} MATCHES "x86_64" AND NOT CMAKE_OSX_ARCHITECTURES)

if(NOT TARGET xsimd)
find_package(xsimd REQUIRED CONFIG)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

that's a very neat trick, <3

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I did not make it up, it comes up frequently in CMake files :)

@serge-sans-paille serge-sans-paille merged commit 8d6d1f1 into xtensor-stack:master Apr 16, 2026
79 of 88 checks passed
@AntoinePrv AntoinePrv deleted the cmake branch April 17, 2026 08:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants