argotorg / argotorg/fe

Implement msg.data

Open
#277 7 comments 0 reactions 0 assignees View on GitHub
type: feature
Dominant language
Rust
Stars
1.7k
Forks
218
Avg merge
1d 7h
Merged PRs (30d)
6

Description

### What is wrong?

We do not yet implement reading `msg.data`

https://github.com/ethereum/fe/blob/177ee35ba4b12150f5cb14152e066d364b1a0161/compiler/src/yul/mappers/expressions.rs#L438

### How can it be fixed

Start here:

https://github.com/ethereum/fe/blob/177ee35ba4b12150f5cb14152e066d364b1a0161/compiler/src/yul/mappers/expressions.rs#L438

Background:

https://docs.soliditylang.org/en/develop/units-and-global-variables.html#block-and-transaction-properties

Contributor guide

No contributing guide indexed for this repository

Research direction

Start at compiler/src/yul/mappers/expressions.rs:438 and read the surrounding Yul expression mapping. Use the linked Solidity block and transaction properties documentation to determine the expected semantics of reading msg.data. Done means msg.data is translated correctly according to that reference.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
blockchain, compilers
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
48/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.