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

Add new required target-c-int-width field to target json #354

Merged
merged 1 commit into from Oct 3, 2017

Conversation

1 participant
@phil-opp
Owner

phil-opp commented Oct 3, 2017

There is a new required field for target configuration files: target-c-int-width. This field controls the size of the argc parameter and the return type of the native main function in std applications. (For no_std applications/libraries, we don't really need this field.) We set the value to 32, since it's the size of an int in most C calling conventions (including the 64-bit Linux calling convention).

See rust-lang/rust#44906 for more details.

Fixes #353

@phil-opp phil-opp merged commit 82c4a6f into master Oct 3, 2017

1 of 3 checks passed

continuous-integration/travis-ci/pr The Travis CI build failed
Details
continuous-integration/travis-ci/push The Travis CI build failed
Details
deploy/netlify Deploy preview ready!
Details

@phil-opp phil-opp deleted the p1 branch Oct 3, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment