-
Notifications
You must be signed in to change notification settings - Fork 46
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
[Experiment] [BROKEN] Make stored captures trivial #686
base: main
Are you sure you want to change the base?
Conversation
This gives us a somewhat robust (given our benchmark suite's variability) perf improvement of a few percent typically. In its current state, this code breaks support for value captures and that would need to be fixed by setting the
Beyond this, we could also improve the struct's layout, but that's a much lower-impact change for further down the line. |
_StoredCapture
stored anAny?
, which is rarely set to anything non-nil
, but since it could it is non-trivial. This experiment tries to see the perf overhead of this non-triviality.