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
runtime: add 24 byte malloc size class #17633
Comments
Naively, it's a one-liner in msize.go:initSizes
but make.bash then crashes. |
Time for 1.8 is now down to only a couple of more working days and while On Thu, Oct 27, 2016 at 3:10 PM, dr2chase notifications@github.com wrote:
|
Fair enough. Here's one data point from the compiler for 1.9 about the prevalence of slice allocations: While compiling std+cmd, 3.42% of all allocations were |
My hunch would be that this is violating some of the assumptions of |
Is this a dup of #8885? |
Yes. Closing this one, since #8885 predates it so much. (Normally we close the dup with the least conversation if the dates are roughly equal) |
We have a 16 and a 32 byte size class, but no 24. Slices escape a lot (io.Reader calls, sort.Sort calls, etc.). We should only allocate 24 bytes for them when they do.
Note sure how best to work this into our current size generation scheme.
@randall77 @bradfitz @aclements
Tentatively marking Go 1.8 since it's seems like a cheap, easy, safe win.
The text was updated successfully, but these errors were encountered: