user0/rust
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Auto merge of #52906 - RalfJung:jemalloc, r=alexcrichton

Browse source 
enable jemalloc assertions when configured to do so

This is essentially a re-submission of the functional part of https://github.com/rust-lang/rust/pull/43648. I was unable to reproduce the issue I had back then, maybe something changed somewhere to no longer trigger the assertion.

Fixes https://github.com/rust-lang/rust/issues/44152
This commit is contained in:
bors 2018-08-02 10:28:42 +00:00
commit 76aeeefdc2
1 changed files with 4 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

9
src/liballoc_jemalloc/build.rs
View file

@ -105,11 +105,10 @@ fn main() {
cmd.arg("--with-jemalloc-prefix=je_");
}
// FIXME: building with jemalloc assertions is currently broken.
// See <https://github.com/rust-lang/rust/issues/44152>.
//if cfg!(feature = "debug") {
// cmd.arg("--enable-debug");
//}
if cfg!(feature = "debug") {
// Enable jemalloc assertions.
cmd.arg("--enable-debug");
}
cmd.arg(format!("--host={}", build_helper::gnu_target(&target)));
cmd.arg(format!("--build={}", build_helper::gnu_target(&host)));