Skip to content

Commit

Permalink
Auto merge of #45163 - sfackler:jemalloc-5, r=<try>
Browse files Browse the repository at this point in the history
Upgrade jemalloc to 5.0.1

The build script needs to run autoconf which should be fine, right?

We're building against 5.0.1 with jemalloc/jemalloc@0a4f5a7 cherry-picked on top.

r? @alexcrichton
  • Loading branch information
bors committed Oct 12, 2017
2 parents fbb5054 + 1d78cc2 commit 116157c
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 7 deletions.
4 changes: 2 additions & 2 deletions .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ matrix:
if: type = pull_request OR branch = auto

- env: IMAGE=dist-x86_64-linux DEPLOY=1
if: branch = try OR branch = auto
if: OR branch = auto

# "alternate" deployments, these are "nightlies" but don't have assertions
# turned on, they're deployed to a different location primarily for projects
Expand Down Expand Up @@ -120,7 +120,7 @@ matrix:
- env: IMAGE=dist-aarch64-linux DEPLOY=1
if: branch = auto
- env: IMAGE=dist-android DEPLOY=1
if: branch = auto
if: branch = auto OR branch = try
- env: IMAGE=dist-arm-linux DEPLOY=1
if: branch = auto
- env: IMAGE=dist-armhf-linux DEPLOY=1
Expand Down
2 changes: 1 addition & 1 deletion src/jemalloc
Submodule jemalloc updated 259 files
10 changes: 6 additions & 4 deletions src/liballoc_jemalloc/build.rs
Original file line number Diff line number Diff line change
Expand Up @@ -69,6 +69,10 @@ fn main() {
let cflags = compiler.args()
.iter()
.map(|s| s.to_str().unwrap())
.filter(|&s| {
// separate function/data sections trigger errors with android's TLS emulation
!target.contains("android") || (s != "-ffunction-sections" && s != "-fdata-sections")
})
.collect::<Vec<_>>()
.join(" ");

Expand All @@ -78,6 +82,7 @@ fn main() {
.unwrap()
.replace("C:\\", "/c/")
.replace("\\", "/"))
.arg("--disable-cxx")
.current_dir(&native.out_dir)
.env("CC", compiler.path())
.env("EXTRA_CFLAGS", cflags.clone())
Expand All @@ -93,9 +98,7 @@ fn main() {
.env("AR", &ar)
.env("RANLIB", format!("{} s", ar.display()));

if target.contains("ios") {
cmd.arg("--disable-tls");
} else if target.contains("android") {
if target.contains("android") {
// We force android to have prefixed symbols because apparently
// replacement of the libc allocator doesn't quite work. When this was
// tested (unprefixed symbols), it was found that the `realpath`
Expand All @@ -106,7 +109,6 @@ fn main() {
// If the test suite passes, however, without symbol prefixes then we
// should be good to go!
cmd.arg("--with-jemalloc-prefix=je_");
cmd.arg("--disable-tls");
} else if target.contains("dragonfly") || target.contains("musl") {
cmd.arg("--with-jemalloc-prefix=je_");
}
Expand Down

0 comments on commit 116157c

Please sign in to comment.