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

Generically implement fn horizontal_{and,or,xor} #199

Merged
merged 1 commit into from
Nov 24, 2021

Conversation

workingjubilee
Copy link
Member

20kb off, and it makes the docs significantly less crowded.
Implementation here depends on both Self and T implementing the Bit{Op} trait in question (so that it's coherent to reason about it as equivalent to a manual implementation where it does a series of vector Bit{Op}s and then a final fold of Bit{Op}s on essentially Iterator<Item=T>).

@workingjubilee
Copy link
Member Author

...well, that didn't happen locally.

@workingjubilee
Copy link
Member Author

workingjubilee commented Nov 22, 2021

I do not think that segfault is the fault of this PR's code, because we've been having random segfaults on rust-lang/rust also (see rust-lang/rust#90812) and it didn't even get as far as running the tests. ( Also rerunning it doesn't repeat it. )

@workingjubilee workingjubilee changed the title Generically implement horizontal_{and,or,xor} Generically implement horizontal_{and,or,xor} Nov 22, 2021
@workingjubilee workingjubilee changed the title Generically implement horizontal_{and,or,xor} Generically implement fn horizontal_{and,or,xor} Nov 24, 2021
Copy link
Member

@programmerjake programmerjake left a comment

Choose a reason for hiding this comment

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

also lgtm

@programmerjake programmerjake mentioned this pull request Nov 24, 2021
@workingjubilee workingjubilee merged commit ae61210 into master Nov 24, 2021
@calebzulawski calebzulawski deleted the horizontal-bitops branch October 17, 2022 00:19
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