0xMiden / 0xMiden/protocol

Optimize input note num inputs/assets memory layout

Đang mở
#2,256 2 bình luận 0 reaction 1 người được giao Được @hawkadrian nhận Xem trên GitHub
good first issue kernels
Ngôn ngữ chính
Rust
Star
132
Fork
167
Merge trung bình
1 ngày 23 giờ
Pull request đã merge (30 ngày)
110

Mô tả

After #2109 is closed (to avoid merge conflicts), we should be able to make a tiny PR that moves these two constants closer together:

https://github.com/0xMiden/miden-base/blob/eee0e8450b825857e7d56ffebfaa269378a61aca/crates/miden-protocol/src/transaction/kernel/memory.rs#L388-L389

It should be possible to do:
```rust
pub const INPUT_NOTE_NUM_INPUTS_OFFSET: MemoryOffset = 36;
pub const INPUT_NOTE_NUM_ASSETS_OFFSET: MemoryOffset = 37;
pub const INPUT_NOTE_ASSETS_OFFSET: MemoryOffset = 40
```

This applies to `memory.masm` in the same way.

Context: https://github.com/0xMiden/miden-base/pull/2249#discussion_r2679280006

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Đánh giá

Issue này chưa được đánh giá.

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.