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

Remove some redundant parts from unrolled_find_u16s #74289

Merged
merged 3 commits into from
Aug 8, 2020

Conversation

tesuji
Copy link
Contributor

@tesuji tesuji commented Jul 13, 2020

See each commit message for details.

r? @wesleywiser from old PR #67705 .

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jul 13, 2020
@tesuji
Copy link
Contributor Author

tesuji commented Jul 13, 2020

A very small improvement in codegen with opt-level <=2: https://rust.godbolt.org/z/8fMGav .

@tesuji

This comment has been minimized.

@bors
Copy link
Contributor

bors commented Jul 28, 2020

☔ The latest upstream changes (presumably #73265) made this pull request unmergeable. Please resolve the merge conflicts.

@tesuji
Copy link
Contributor Author

tesuji commented Jul 31, 2020

r? @LukasKalbertodt for review or reassignment

Copy link
Member

@LukasKalbertodt LukasKalbertodt left a comment

Choose a reason for hiding this comment

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

Looks like a nice improvement :)
Thanks!

@LukasKalbertodt
Copy link
Member

As this is platform dependent code, I assume this will not do anything about compiler performance? So I will not rollup=never, right? If I'm wrong just tell me or bors ^_^

@bors r+

@bors
Copy link
Contributor

bors commented Aug 8, 2020

📌 Commit f55e4d0 has been approved by LukasKalbertodt

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Aug 8, 2020
@bors
Copy link
Contributor

bors commented Aug 8, 2020

⌛ Testing commit f55e4d0 with merge c989ac1...

@bors
Copy link
Contributor

bors commented Aug 8, 2020

☀️ Test successful - checks-actions, checks-azure
Approved by: LukasKalbertodt
Pushing c989ac1 to master...

@bors bors added the merged-by-bors This PR was explicitly merged by bors. label Aug 8, 2020
@bors bors merged commit c989ac1 into rust-lang:master Aug 8, 2020
@tesuji tesuji deleted the unroll branch August 8, 2020 13:48
@cuviper cuviper added this to the 1.47.0 milestone May 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merged-by-bors This PR was explicitly merged by bors. S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants