-
Notifications
You must be signed in to change notification settings - Fork 321
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
v2.0: feat: deplete compute meter for vm errors (backport of #3751) #3956
Conversation
Cherry-pick of b120fc1 has failed:
To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally |
The Firedancer team maintains a line-for-line reimplementation of the |
2bc37de
to
9f7d415
Compare
2326781
to
b1c9268
Compare
* feat: deplete compute meter for vm errors * add tests * rekey apply_cost_tracker_during_replay (cherry picked from commit b120fc1) # Conflicts: # programs/sbf/tests/programs.rs # sdk/src/feature_set.rs
3e82a25
to
3e692ee
Compare
this needs sme approval |
Closing in favor of #4296 |
Problem
Implements SIMD-0182 by making sure that programs that fail with VM errors consume the remaining compute budget to simplify compute unit handling.
Summary of Changes
Feature Gate Issue: solana-labs#29595
This is an automatic backport of pull request #3751 done by [Mergify](https://mergify.com).