-
Notifications
You must be signed in to change notification settings - Fork 13k
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
enable jemalloc assertions when configured to do so #52906
Conversation
(rust_highfive has picked a reviewer for you, use r? to override) |
@gnzlbg says nothing should have changed. I tried these settings:
That should be enough to properly enable this, right? Then I did I verified that the relevant code in |
Yes, this change should be enough to configure If you want to be extra sure that it is being compiled with debug assertions enabled, building it with This line of the output: https://travis-ci.org/jemalloc/jemalloc/jobs/408750420#L346 should contain |
@bors: r+ Ok! Let's see if it gets past CI |
📌 Commit 872395a has been approved by |
☀️ Test successful - status-appveyor, status-travis |
This is essentially a re-submission of the functional part of #43648. I was unable to reproduce the issue I had back then, maybe something changed somewhere to no longer trigger the assertion.
Fixes #44152