Skip to content

http: avoid some copies#642

Merged
arnetheduck merged 2 commits intomasterfrom
avoidcopies
Apr 9, 2026
Merged

http: avoid some copies#642
arnetheduck merged 2 commits intomasterfrom
avoidcopies

Conversation

@arnetheduck
Copy link
Copy Markdown
Member

While waiting for:

In particular, the move-on-return are ugly but needed for #639 to do its job well

@arnetheduck
Copy link
Copy Markdown
Member Author

Pre:

| Small/small    |    0.059s |   1000 | 16830.970 |    0.031 MB |    0.016 MB |    0.514 MB/s |    0.273 MB/s |
| Medium/small   |    0.783s |   1000 | 1276.412 | 1000.000 MB |    0.021 MB | 1276.412 MB/s |    0.027 MB/s |
| Small/Medium   |    0.993s |   1000 | 1007.081 |    0.031 MB | 1000.000 MB |    0.031 MB/s | 1007.081 MB/s |
| Medium/Medium  |    1.475s |   1000 |  677.819 | 1000.000 MB | 1000.000 MB |  677.819 MB/s |  677.819 MB/s |

Post:

| Small/small    |    0.061s |   1000 | 16490.837 |    0.031 MB |    0.016 MB |    0.503 MB/s |    0.267 MB/s |
| Medium/small   |    0.760s |   1000 | 1315.258 | 1000.000 MB |    0.021 MB | 1315.258 MB/s |    0.028 MB/s |
| Small/Medium   |    0.919s |   1000 | 1088.442 |    0.031 MB | 1000.000 MB |    0.033 MB/s | 1088.442 MB/s |
| Medium/Medium  |    1.353s |   1000 |  738.962 | 1000.000 MB | 1000.000 MB |  738.962 MB/s |  738.962 MB/s |

@arnetheduck
Copy link
Copy Markdown
Member Author

arnetheduck commented Apr 8, 2026

Avoiding nim-lang/Nim#25063:

| Medium/Medium  |    1.093s |   1000 |  914.615 | 1000.000 MB | 1000.000 MB |  914.615 MB/s |  914.615 MB/s |

removed @ change from this branch (it needs more work)

@arnetheduck arnetheduck marked this pull request as draft April 8, 2026 22:20
While waiting for:

* #601
* #578
* nim-lang/Nim#25057

In particular, the move-on-return are ugly but needed for #639 to do its
job well
@arnetheduck arnetheduck marked this pull request as ready for review April 9, 2026 06:41
@arnetheduck arnetheduck merged commit 3bc3d80 into master Apr 9, 2026
26 checks passed
@arnetheduck arnetheduck deleted the avoidcopies branch April 9, 2026 15:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants