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

Remove jemalloc from the runtime #9933

Merged
merged 1 commit into from
Oct 18, 2013
Merged

Commits on Oct 18, 2013

  1. Remove jemalloc from the runtime

    As discovered in rust-lang#9925, it turns out that we weren't using jemalloc on most
    platforms. Additionally, on some platforms we were using it incorrectly and
    mismatching the libc version of malloc with the jemalloc version of malloc.
    
    Additionally, it's not clear that using jemalloc is indeed a large performance
    win in particular situtations. This could be due to building jemalloc
    incorrectly, or possibly due to using jemalloc incorrectly, but it is unclear at
    this time.
    
    Until jemalloc can be confirmed to integrate correctly on all platforms and has
    verifiable large performance wins on platforms as well, it shouldn't be part of
    the default build process. It should still be available for use via the
    LD_PRELOAD trick on various architectures, but using it as the default allocator
    for everything would require guaranteeing that it works in all situtations,
    which it currently doesn't.
    
    Closes rust-lang#9925
    alexcrichton committed Oct 18, 2013
    5 Configuration menu
    Copy the full SHA
    90911d7 View commit details
    Browse the repository at this point in the history