Join GitHub today
GitHub is home to over 50 million developers working together to host and review code, manage projects, and build software together.
Sign upUpdate jemalloc-sys and unfork jemallocator #20645
Open
Comments
|
Also being discussed at gnzlbg/jemallocator#34 (comment) |
bors-servo
added a commit
that referenced
this issue
May 30, 2018
Update to latest jemallocator and jemalloc-sys Updates the allocator component to use the latest jemallocator. cc @SimonSapin - [ ] `./mach build -d` does not report any errors - [ ] `./mach test-tidy` does not report any errors - [x] These changes fix #20645 - [x] There are tests for these changes: everything that allocates memory tests them. <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/20848) <!-- Reviewable:end -->
bors-servo
added a commit
that referenced
this issue
May 30, 2018
Update to latest jemallocator and jemalloc-sys Updates the allocator component to use the latest jemallocator. cc @SimonSapin - [ ] `./mach build -d` does not report any errors - [ ] `./mach test-tidy` does not report any errors - [x] These changes fix #20645 - [x] There are tests for these changes: everything that allocates memory tests them. <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/20848) <!-- Reviewable:end -->
bors-servo
added a commit
that referenced
this issue
May 30, 2018
Update to latest jemallocator and jemalloc-sys Updates the allocator component to use the latest jemallocator. cc @SimonSapin - [ ] `./mach build -d` does not report any errors - [ ] `./mach test-tidy` does not report any errors - [x] These changes fix #20645 - [x] There are tests for these changes: everything that allocates memory tests them. <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/20848) <!-- Reviewable:end -->
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
In #20641 we duplicated ~70 lines from the
jemallocatorcrate in order to be able to keep using an orderjemalloc-sys, because the new one doesn’t build on our Android infra:/home/servo/.cargo/registry/src/github.com-1ecc6299db9ec823/jemalloc-sys-0.1.5/jemalloc/src/background_thread.c:83:2: error: unknown type name 'cpu_set_t'Upstream issue: jemalloc/jemalloc#1175