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

Reenable jemalloc #7217

Closed
brson opened this issue Jun 18, 2013 · 3 comments
Closed

Reenable jemalloc #7217

brson opened this issue Jun 18, 2013 · 3 comments
Labels
A-runtime Area: std's runtime and "pre-main" init for handling backtraces, unwinds, stack overflows

Comments

@brson
Copy link
Contributor

brson commented Jun 18, 2013

I turned it off because it causes canary kills in compile-fail tests.

@alexcrichton
Copy link
Member

Nominating production-ready.

@bstrie
Copy link
Contributor

bstrie commented Jul 23, 2013

I think that @thestinger was concerned that, as long as split stacks are supported (not just enabled-by-default, just supported at all), jemalloc cannot be used with Rust. Is this correct?

@brson
Copy link
Contributor Author

brson commented Aug 10, 2013

@bstrie I can't imagine why that would be true.

I think we should turn on jemalloc now while split stacks are disabled.

bors added a commit that referenced this issue Aug 20, 2013
flip1995 pushed a commit to flip1995/rust that referenced this issue May 20, 2021
…o-alias, r=flip1995

Add `cargo collect-metadata` as an cargo alias for the metadata collection lint

This PR adds a new alias to run the metadata collection monster on `clippy_lints`. I'm currently using it to create the `metadata_collection.json` file and I plan to use it in the `deply.sh` script. Having it as a new alias enables us to simply use:

```sh
cargo collect-metadata
```

It sometimes requires running `cargo clean` before collecting the metadata due to caching. I'm still debating if I should include a cargo clean as part of the `run_metadata_collection_lint` test or not. Input on this would be greatly appreciated 🙃

That's it, just a small change that can be reviewed and merged in parallel to rust-lang#7214.

---

See: rust-lang#7172 for the full metadata collection to-do list or to suggest a new feature in connection to it.

---

changelog: none

r? `@flip1995` btw. feel free to pass these PRs one to other team members as well if you want.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-runtime Area: std's runtime and "pre-main" init for handling backtraces, unwinds, stack overflows
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants