Skip to content

Define arch for iOS/ARM #11472

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

Merged
merged 1 commit into from
Jan 11, 2014
Merged

Define arch for iOS/ARM #11472

merged 1 commit into from
Jan 11, 2014

Conversation

kud1ing
Copy link

@kud1ing kud1ing commented Jan 11, 2014

This fixes #11336

I guess the type sizes are correct for both OS X and iOS, but i am not certain.
In any case, i'd rather have any iOS build at all, so that we have something to improve upon.

@@ -1160,6 +1160,7 @@ pub mod types {
}
}

#[cfg(target_arch = "arm")]
Copy link
Contributor

Choose a reason for hiding this comment

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

Does Android need it too?

Copy link
Author

Choose a reason for hiding this comment

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

Android has some arm/x86 switches in this file already.

Copy link
Contributor

Choose a reason for hiding this comment

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

Ok, but I think that targetting on ARM with target_arch = "arm" will trigger it for Android too, no? Does that have any side effect?

Copy link
Author

Choose a reason for hiding this comment

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

The change is within a #[cfg(target_os = "macos")] block.

Copy link
Contributor

Choose a reason for hiding this comment

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

Right!

Copy link
Author

Choose a reason for hiding this comment

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

It find it easy to get lost in this file.

bors added a commit that referenced this pull request Jan 11, 2014
This fixes #11336

I guess the type sizes are correct for both OS X and iOS, but i am not certain.
In any case, i'd rather have any iOS build at all, so that we have something to improve upon.
@bors bors closed this Jan 11, 2014
@bors bors merged commit 6bbd2ed into rust-lang:master Jan 11, 2014
@kud1ing kud1ing deleted the iOS branch January 12, 2014 09:37
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.

Can't compile "libc.rs" for iOS
3 participants