vm, runtime: implement direct mapping supercharged#6332
Merged
topointon-jump merged 1 commit intomainfrom Oct 15, 2025
Merged
Conversation
c4ba46a to
507a56d
Compare
e80f90e to
596896f
Compare
5505ff3 to
c5675b1
Compare
dce0041 to
58301df
Compare
Performance Measurements ⏳
|
Performance Measurements ⏳
|
Performance Measurements ⏳
|
Performance Measurements ⏳
|
Performance Measurements ⏳
|
Performance Measurements ⏳
|
0be0250 to
6492736
Compare
Performance Measurements ⏳
|
1de66fe to
d6c4d05
Compare
Performance Measurements ⏳
|
Performance Measurements ⏳
|
3d022c3 to
4646ef1
Compare
13a8f54 to
3eb4b41
Compare
Performance Measurements ⏳
|
3eb4b41 to
e7c5498
Compare
Performance Measurements ⏳
|
e7c5498 to
afc8e8a
Compare
Performance Measurements ⏳
|
Contributor
ibhatt-jumptrading
left a comment
There was a problem hiding this comment.
logic seems good to me, it would be nice if you added comments for the different cases in bpf_serlalize and cpi_common explaining what they do/are.
Also worth combing over for style, I didn't point out every style issue
Performance Measurements ⏳
|
d5148ba to
1152428
Compare
Performance Measurements ⏳
|
1152428 to
8a27f67
Compare
Performance Measurements ⏳
|
8a27f67 to
f8becfb
Compare
Performance Measurements ⏳
|
mjain-jump
reviewed
Oct 15, 2025
f8becfb to
3b10a4f
Compare
ibhatt-jumptrading
approved these changes
Oct 15, 2025
Performance Measurements ⏳
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Implements both the the latest incarnation of
account_data_direct_mapping(anza-xyz/agave#5871) and the tightly-coupledstricter_abi_and_runtime_constraints(anza-xyz/agave#7113) feature gates.stricter_abi_and_runtime_constraintsmust be activated beforeaccount_data_direct_mapping. Therefore all fuzzing, ledgers and unit tests withaccount_data_direct_mappingactivated also activatestricter_abi_and_runtime_constraints.Both feature gates have been extensively fuzzed and tested. Ledgers for each have been added to CI. Fuzz fixtures have been added to test vectors. The vm unit tests have been updated to reflect the new behaviour.