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 i8mm feature with bootstrap compiler. #1252

Merged
merged 1 commit into from
Nov 15, 2021

Conversation

ehuss
Copy link
Contributor

@ehuss ehuss commented Nov 14, 2021

The beta compiler (1.57) does not support the i8mm feature. This fixes a build failure by not enabling it when bootstrapping.

@rust-highfive
Copy link

r? @Amanieu

(rust-highfive has picked a reviewer for you, use r? to override)

@ehuss
Copy link
Contributor Author

ehuss commented Nov 14, 2021

I'm not certain that this will work, as I do not have an arm system to test on.

@SparrowLii
Copy link
Member

If we accept this change, does it mean that we need to change it again when the beta compiler supports the i8mm feature?

@ehuss
Copy link
Contributor Author

ehuss commented Nov 15, 2021

Yes, I believe that is what is usually done (#1150 for example).

@Amanieu Amanieu merged commit 05e8072 into rust-lang:master Nov 15, 2021
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.

4 participants