Skip to content

feat: Add batch::to_native#1363

Merged
JohanMabille merged 1 commit into
xtensor-stack:masterfrom
AntoinePrv:raw
Jun 9, 2026
Merged

feat: Add batch::to_native#1363
JohanMabille merged 1 commit into
xtensor-stack:masterfrom
AntoinePrv:raw

Conversation

@AntoinePrv

@AntoinePrv AntoinePrv commented Jun 4, 2026

Copy link
Copy Markdown
Contributor

Using batch::register_type can sometimes be involved if it is heavily templated or if it is deducted from an auto const&. This makes casting to a native type when needed easier.

It should not be necessary to explicitly cast most of the time, but as we know with various compiler and intrinsicts implementations, we do what we can.

@AntoinePrv AntoinePrv force-pushed the raw branch 2 times, most recently from ce4c360 to 6830b4b Compare June 4, 2026 13:26
@serge-sans-paille

Copy link
Copy Markdown
Contributor

I'm not against the idea, but I pretty much dislike the name which is not explicit enough to me :-)
Suggestions: native_register, raw_register, get_register, read_register, native, to_native...
I have a slight preference for the one including the native part

@AntoinePrv AntoinePrv changed the title feat: Add batch::raw feat: Add batch::to_native Jun 4, 2026
@AntoinePrv

Copy link
Copy Markdown
Contributor Author

Done and added a some doc on "Breaking out of xsimd".

@JohanMabille

Copy link
Copy Markdown
Member

Let's get this one in!

@JohanMabille JohanMabille merged commit 01f17ec into xtensor-stack:master Jun 9, 2026
89 checks passed
@AntoinePrv AntoinePrv deleted the raw branch June 9, 2026 15:34
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.

3 participants