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

Fix unused link_name attribute. #1251

Merged
merged 1 commit into from
Nov 13, 2021
Merged

Conversation

ehuss
Copy link
Contributor

@ehuss ehuss commented Nov 13, 2021

It looks like there was a mistake commenting out this function, which left an unused attribute.

Unblocks rust-lang/rust#88681 which is extending the unused_attributes lint to catch more situations.

@rust-highfive
Copy link

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @Amanieu (or someone else) soon.

Please see the contribution instructions for more information.

@Amanieu Amanieu merged commit 63a720c into rust-lang:master Nov 13, 2021
@ehuss
Copy link
Contributor Author

ehuss commented Nov 13, 2021

@Amanieu Is it OK for me to bump the stdarch module on rust-lang/rust?

@Amanieu
Copy link
Member

Amanieu commented Nov 13, 2021

Yes, that's fine.

bors added a commit to rust-lang-ci/rust that referenced this pull request Nov 20, 2021
Update stdarch

5 commits in 815d55c..cfba59f
2021-11-08 00:58:47 +0000 to 2021-11-19 01:29:04 +0000
- Work-around buggy Intel chips erroneously reporting BMI1/BMI2 support (rust-lang/stdarch#1249)
- complete armv8 instructions (rust-lang/stdarch#1256)
- Fix i8mm feature with bootstrap compiler. (rust-lang/stdarch#1252)
- Fix unused link_name attribute. (rust-lang/stdarch#1251)
- Add remaining insturctions (rust-lang/stdarch#1250)
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