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

Use @silversquirl's binned allocator #1134

Merged
merged 3 commits into from
Jun 4, 2023
Merged

Conversation

SuperAuguste
Copy link
Member

No description provided.

@SuperAuguste SuperAuguste added the pr:fuzz Attach to a PR to start fuzzing / continually fuzz (please do this before merging!) label Apr 16, 2023
@Techatrix
Copy link
Member

I would suggest to use the GPA in debug builds by default.

@Jarred-Sumner
Copy link
Contributor

Jarred-Sumner commented Apr 18, 2023

using c_allocator fixes #1128

haven't tried with this

#889 (comment)

@SuperAuguste
Copy link
Member Author

Took your suggestion and applied it, sorry this took so long!

@Jarred-Sumner
Copy link
Contributor

can you merge this

@SuperAuguste SuperAuguste merged commit 4f0762a into master Jun 4, 2023
7 checks passed
@SuperAuguste SuperAuguste deleted the use-binned-allocator branch June 4, 2023 06:43
acristoffers pushed a commit to acristoffers/zls that referenced this pull request Jun 5, 2023
@Jarred-Sumner
Copy link
Contributor

thank you

Techatrix pushed a commit that referenced this pull request Aug 26, 2023
* Use @silversquirl's binned allocator

* Finish this
KoltPenny pushed a commit to KoltPenny/zls that referenced this pull request Oct 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr:fuzz Attach to a PR to start fuzzing / continually fuzz (please do this before merging!)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants