Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[arm] vadd, vaddd, vaddq, vaddl #140

Merged
merged 1 commit into from
Oct 26, 2017
Merged

Conversation

gnzlbg
Copy link
Contributor

@gnzlbg gnzlbg commented Oct 22, 2017

No description provided.

@gnzlbg gnzlbg force-pushed the neon_intrinsics branch 4 times, most recently from 7083d74 to 8682779 Compare October 22, 2017 21:48
(i32x2, i64x2, as_i64x2),
(u8x8, u16x8, as_u16x8),
(u16x4, u32x4, as_u32x4),
(u32x2, u64x2, as_u64x2)
Copy link
Member

Choose a reason for hiding this comment

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

Oh I thought these were only for sem-width casts? What are the semantics of the different-width casts? Does each lane just get promoted?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Does each lane just get promoted?

Yes, that's what simd_cast does and what I intended. @BurntSushi how did you envision this? The number of as_... functions is going to explode if I keep doing this for the arm intrinsics. I would be fine with it, but maybe we should consider providing simd_cast as part of the library API.

@gnzlbg
Copy link
Contributor Author

gnzlbg commented Oct 25, 2017

@BurntSushi @alexcrichton I've filled #151 to discuss this. Right now this blocks the aarch64 refactor of #139 .

@BurntSushi
Copy link
Member

@alexcrichton and I talked about this in person and we're good with this.

@BurntSushi BurntSushi merged commit f08e9c2 into rust-lang:master Oct 26, 2017
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