Skip to content

Fix static_assert on i8mm<neon64>#1187

Merged
serge-sans-paille merged 1 commit into
xtensor-stack:masterfrom
AntoinePrv:fix-assert
Oct 31, 2025
Merged

Fix static_assert on i8mm<neon64>#1187
serge-sans-paille merged 1 commit into
xtensor-stack:masterfrom
AntoinePrv:fix-assert

Conversation

@AntoinePrv
Copy link
Copy Markdown
Contributor

This failed locally on i8mm<neon64>.
Not sure this is the best way to express it.

@serge-sans-paille
Copy link
Copy Markdown
Contributor

That's totally fine. Could you add validation for in on our github ci, or do you want me to handle it an other commit?

@AntoinePrv
Copy link
Copy Markdown
Contributor Author

I'm on an Apple M3 here using Clang 20 from conda-forge (so NOT AppleClang).
I'm surprised this was not cached in the CI so not sure where it is coming from.
I have a -march=native that shows up in my compilation flags, that's probably where it comes from.

@serge-sans-paille serge-sans-paille merged commit 0fd437f into xtensor-stack:master Oct 31, 2025
59 checks passed
@AntoinePrv AntoinePrv deleted the fix-assert branch October 31, 2025 14:51
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