Allocate some query results on an arena#70161
Conversation
|
The job Click to expand the log.I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
|
☔ The latest upstream changes (presumably #70211) made this pull request unmergeable. Please resolve the merge conflicts. |
|
Rebased. |
|
☔ The latest upstream changes (presumably #70204) made this pull request unmergeable. Please resolve the merge conflicts. |
|
Ping from triage: |
|
@rustbot modify labels: +S-waiting-on-review, -S-waiting-on-author |
|
☔ The latest upstream changes (presumably #70536) made this pull request unmergeable. Please resolve the merge conflicts. |
davidtwco
left a comment
There was a problem hiding this comment.
Sorry for the delay in reviewing, this looks good to me but I think someone else should take a look too, might warrant a perf run too?
|
r? @Zoxc |
|
☔ The latest upstream changes (presumably #70692) made this pull request unmergeable. Please resolve the merge conflicts. |
|
☔ The latest upstream changes (presumably #70156) made this pull request unmergeable. Please resolve the merge conflicts. |
|
Can I get a perf run please? |
|
@bors try @rust-timer queue |
|
Awaiting bors try build completion |
|
⌛ Trying commit 802c0be with merge 8275901066c5f9ba900b15a5456e19883ac89893... |
|
☀️ Try build successful - checks-azure |
|
Queued 8275901066c5f9ba900b15a5456e19883ac89893 with parent 607b858, future comparison URL. |
|
Finished benchmarking try commit 8275901066c5f9ba900b15a5456e19883ac89893, comparison URL. |
|
Perf seems positive. |
|
📌 Commit 802c0be has been approved by |
eddyb
left a comment
There was a problem hiding this comment.
All of this makes sense, thanks!
|
⌛ Testing commit 802c0be with merge a29bd4a433c167b7790bf691ff37d604a04f1955... |
|
@bors retry yield |
|
☀️ Test successful - checks-azure |
This avoids a cloning few
LrcandVecs in the queries.