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 "-arch arm64" flag for aarch64-ios-sim #759

Merged
merged 1 commit into from
Jan 19, 2023
Merged

Conversation

gh2o
Copy link
Contributor

@gh2o gh2o commented Dec 1, 2022

Passing "-arch arm64" as a single string to clang causes an "argument unused during compilation" warning that becomes an error if -Werror is enabled. Fix passes it as two separate arguments (unfortunately "-arch=arm64" does not work either).

Passing "-arch arm64" as a single string to clang causes an "argument
unused during compilation" warning that becomes an error if -Werror is
enabled. Fix passes it as two separate arguments (unfortunately
"-arch=arm64" does not work either).
@gh2o
Copy link
Contributor Author

gh2o commented Jan 19, 2023

Bump :)

Copy link
Member

@thomcc thomcc left a comment

Choose a reason for hiding this comment

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

Oh yeah, that seems completely wrong

@thomcc thomcc merged commit cf0a78b into rust-lang:main Jan 19, 2023
linabutler added a commit to mozilla/application-services that referenced this pull request Oct 13, 2023
This version contains rust-lang/cc-rs#759, which fixes our M1 iOS
Simulator megazord builds.
linabutler added a commit to mozilla/application-services that referenced this pull request Oct 13, 2023
This version contains rust-lang/cc-rs#759, which fixes our M1 iOS
Simulator megazord builds.
github-merge-queue bot pushed a commit to mozilla/application-services that referenced this pull request Oct 13, 2023
This version contains rust-lang/cc-rs#759, which fixes our M1 iOS
Simulator megazord builds.
bzhoek added a commit to bzhoek/syntact that referenced this pull request Dec 15, 2023
bzhoek added a commit to bzhoek/syntact that referenced this pull request Dec 16, 2023
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

2 participants