wasm baseline compiler: increase some Vector inline-capacity sizes
Categories
(Core :: JavaScript: WebAssembly, enhancement)
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox68 | --- | fixed |
People
(Reporter: jseward, Assigned: jseward)
Details
Attachments
(1 file, 2 obsolete files)
|
2.91 KB,
patch
|
Details | Diff | Splinter Review |
Profiling with DHAT of wasm baseline compiling the Tanks demo shows some
opportunities for reducing the amount of heap allocation, by modestly
increasing the inline-capacity values for a few Vector types.
| Assignee | ||
Comment 1•6 years ago
|
||
With the patch from bug 1532592 applied before this one, for compiling the
Tanks demo on an x86_64-linux build, gcc 8.2.1, -O2, this patch reduces the
amount of heap allocation from 156,924,761 bytes in 60,767 blocks to
155,993,057 bytes in 48,713 blocks. The instruction count falls from 2379
million to 2377 million.
Taking together this patch and the patch from bug 1532592, the number of heap
blocks is reduced to 51.5% of what it was, and total allocation to 69.0% of
what it was. Instruction count is down 0.7%.
| Assignee | ||
Comment 2•6 years ago
|
||
| Assignee | ||
Comment 3•6 years ago
|
||
Profiling with DHAT of wasm baseline compiling the Tanks demo shows some
opportunities for reducing the amount of heap allocation, by modestly
increasing the inline-capacity values for a few Vector types.
Compiling the Tanks demo on x86_64-linux, this patch reduces the number of
allocated blocks by around 12000, whilst increasing neither the total
allocated bytes nor the dynamic instruction count.
Comment 4•6 years ago
|
||
There are some r+ patches which didn't land and no activity in this bug for 2 weeks.
:jseward, could you have a look please?
Comment 6•6 years ago
|
||
| bugherder | ||
| Assignee | ||
Updated•6 years ago
|
Updated•6 years ago
|
Description
•