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

Add a 'start' lang item and use it instead of rust_start #5135

Merged
merged 1 commit into from
Feb 27, 2013

Conversation

brson
Copy link
Contributor

@brson brson commented Feb 27, 2013

r?
#3406

Pretty straightforward. I'm using opaque pointers instead trying to get trans and core to agree on the types of the main function and crate map. One oddity is that this required changing the order of the -lrustrt argument to the linker in order to resolve upcall_new_stack. Linkers are mysterious.

bors added a commit that referenced this pull request Feb 27, 2013
r?

#3406

Pretty straightforward. I'm using opaque pointers instead trying to get trans and core to agree on the types of the main function and crate map. One oddity is that this required changing the order of the `-lrustrt` argument to the linker in order to resolve `upcall_new_stack`. Linkers are mysterious.
@bors bors closed this Feb 27, 2013
@bors bors merged commit f432723 into rust-lang:incoming Feb 27, 2013
bors added a commit to rust-lang-ci/rust that referenced this pull request May 2, 2020
Few improvement to `utils::conf` module

* Fix a few typos
* Handle Option<&Path> early
* Use `env::var_os` when possible

changelog: none
bors added a commit to rust-lang-ci/rust that referenced this pull request May 2, 2020
Fix documentation generation for configurable lints

In rust-lang#5135, the configuration macro changed, but the documentation generation script wasn't updated. This PR catches up on this.

[Preview](https://flip1995.github.io/rust-clippy/master/index.html)

r? @Manishearth

changelog: Document configuration options of lints again.
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