-
Notifications
You must be signed in to change notification settings - Fork 115
Labels
kernelsRelated to transaction, batch, or block kernelsRelated to transaction, batch, or block kernels
Milestone
Description
The general rule we have is that the prefix is at the most significant position, i.e. index 0 in a word on the stack or index 3 in a word in memory.
Therefore, for consistency, the memory layout should be:
[nonce, 0, account_id_suffix, account_id_prefix]
When we load it onto the stack (mem_loadw_be) it would then be:
[account_id_prefix, account_id_suffix, 0, nonce]
So, compared to what we have currently on the stack, prefix and suffix are swapped.
The in-memory order is the same as the advice input order, so this change should apply to the native account loading in the prologue and the foreign account loading in the advice inputs creation, as well as some other places like how we put the native asset ID onto the stack, etc.
Originally posted by @PhilippGackstatter in #2408 (comment)
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
kernelsRelated to transaction, batch, or block kernelsRelated to transaction, batch, or block kernels