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.

Sign up for GitHub

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

Jump to bottom

constify is_power_of_two #65628

Closed
wants to merge 1 commit into from
Closed

constify is_power_of_two #65628

wants to merge 1 commit into from

Conversation

tesuji
Copy link
Contributor

@tesuji tesuji commented Oct 20, 2019 •
edited
Loading

See this godbolt link for the difference in debug and optimized build of old and new code.

@tesuji

This comment has been minimized.

Sign in to view
@tesuji
Copy link
Contributor Author

tesuji commented Oct 20, 2019

Looks like scott is busy. r? @sfackler

@rust-highfive rust-highfive assigned sfackler and unassigned scottmcm Oct 20, 2019
@nikic
Copy link
Contributor

nikic commented Oct 20, 2019

Duplicate of #65092?

@tesuji
Copy link
Contributor Author

tesuji commented Oct 20, 2019

Oh. It is. Closing then!

@tesuji tesuji closed this Oct 20, 2019
@tesuji tesuji deleted the const-is_power_of_two branch October 20, 2019 16:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@sfackler sfackler

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@tesuji @nikic @sfackler @scottmcm