Skip to content

Conversation

@nilsdeppe
Copy link
Contributor

Thanks for the great library!

Add the arch namespace needed to get access to avx::batch.

It looks like the README uses batch<double, 4>. If that's preferred I can update the PR to do that instead.

@JohanMabille
Copy link
Member

Thanks for this fix. Both usages are possible (either with the size or with through the arch namespace). A huge refactoring is on-going so that users can specify explicitly the architectur instead of the size.

@JohanMabille JohanMabille merged commit efa5c3e into xtensor-stack:master Jun 2, 2021
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