-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Collapse revision context into specialized context to avoid allocations #11009
Collapse revision context into specialized context to avoid allocations #11009
Conversation
Codecov Report
@@ Coverage Diff @@
## main #11009 +/- ##
=======================================
Coverage 87.97% 87.97%
=======================================
Files 188 188
Lines 9115 9116 +1
=======================================
+ Hits 8019 8020 +1
- Misses 840 841 +1
+ Partials 256 255 -1
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: julz The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Proposed Changes
Always wanted to play with custom value contexts, so here we are! This spares us a bit of linked list lookup and context allocation, so yay!
Release Note
/assign @julz @vagababov