argotorg / argotorg/solidity

Implement the binary search dispatch for the IR codegen

Open
#12,688 4 comments 0 reactions 0 assignees View on GitHub
low effort medium impact must have eventually
Dominant language
C++
Stars
25.7k
Forks
6.2k
Avg merge
2d 19h
Merged PRs (30d)
29

Description

The binary search for the function dispatch seems to be only implemented in the legacy codegen.

Context:
https://github.com/ethereum/solidity/blob/da50176b1ffec4444996cb4551e6a78f99bd2990/libsolidity/codegen/ContractCompiler.cpp#L325

Implement the same for the IR based codegen.

Contributor guide

Open the contributing guide

Research direction

Start with the referenced dispatch logic in libsolidity/codegen/ContractCompiler.cpp around line 325, then locate the corresponding entry point in the IR-based codegen. Compare the two dispatch paths and determine how the existing binary-search behavior should carry over. Done means IR-generated contracts use binary search for function dispatch as the legacy codegen does.

Written by the indexing model from the issue text.

Assessment

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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.