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

Inline static_empty #280

Merged
merged 1 commit into from
Jul 15, 2021
Merged

Inline static_empty #280

merged 1 commit into from
Jul 15, 2021

Conversation

Zoxc
Copy link
Contributor

@Zoxc Zoxc commented Jul 15, 2021

This ended up not being inlined into reserve_rehash.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
@Amanieu
Copy link
Member

Amanieu commented Jul 15, 2021

I would have expected rustc's const-eval to always resolve this, but apparently not.

@bors r+

@bors
Copy link
Contributor

bors commented Jul 15, 2021

📌 Commit e9957e0 has been approved by Amanieu

@bors
Copy link
Contributor

bors commented Jul 15, 2021

⌛ Testing commit e9957e0 with merge d108056...

@bors
Copy link
Contributor

bors commented Jul 15, 2021

☀️ Test successful - checks-actions
Approved by: Amanieu
Pushing d108056 to master...

@bors bors merged commit d108056 into rust-lang:master Jul 15, 2021
@Zoxc Zoxc deleted the inline-static-empty branch July 15, 2021 15:07
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.

None yet

3 participants