Skip to content

ci: Add build tool to workflow#453

Merged
elle-j merged 3 commits intomainfrom
lj/fix-workflow
Jan 15, 2026
Merged

ci: Add build tool to workflow#453
elle-j merged 3 commits intomainfrom
lj/fix-workflow

Conversation

@elle-j
Copy link
Copy Markdown
Contributor

@elle-j elle-j commented Jan 14, 2026

Fixes this and this failed runs.

@elle-j elle-j requested a review from xermicus January 14, 2026 16:59
@elle-j elle-j marked this pull request as draft January 14, 2026 20:38
@github-actions
Copy link
Copy Markdown

Benchmarks

  • Ref Before: main
  • Ref After: lj/fix-workflow

Compile E2E

Results
group                                            resolc_before                          resolc_changes
-----                                            -------------                          --------------
Dependency/resolc                                1.00    195.0±1.19ms        ? ?/sec    1.03    200.3±1.49ms        ? ?/sec
Dependency/solc                                  1.00     77.5±0.40ms        ? ?/sec    1.00     77.8±0.66ms        ? ?/sec
Empty/resolc                                     1.00     75.7±0.71ms        ? ?/sec    1.04     78.5±0.67ms        ? ?/sec
Empty/solc                                       1.00      6.7±0.07ms        ? ?/sec    1.02      6.8±0.08ms        ? ?/sec
LargeDivRem/resolc                               1.00    178.8±1.02ms        ? ?/sec    1.01    181.0±1.00ms        ? ?/sec
LargeDivRem/solc                                 1.00     21.8±0.09ms        ? ?/sec    1.01     22.1±0.16ms        ? ?/sec
Memset (`--yul`)/resolc                          1.00     77.9±1.01ms        ? ?/sec    1.04     80.7±0.94ms        ? ?/sec
Memset (`--yul`)/solc                            1.00      5.6±0.05ms        ? ?/sec    1.02      5.8±0.06ms        ? ?/sec
Multiple Contracts (`--standard-json`)/resolc    1.00       2.7±0.01s        ? ?/sec    1.01       2.7±0.01s        ? ?/sec
Multiple Contracts (`--standard-json`)/solc      1.00    835.4±2.41ms        ? ?/sec    1.01    842.4±3.06ms        ? ?/sec
Return (`--yul`)/resolc                          1.00     70.6±1.02ms        ? ?/sec    1.04     73.1±1.14ms        ? ?/sec
Return (`--yul`)/solc                            1.00      4.1±0.04ms        ? ?/sec    1.04      4.3±0.05ms        ? ?/sec

Parse Yul

Results
group             yul_parse_before                       yul_parse_changes
-----             ----------------                       -----------------
Baseline/parse    1.00     19.0±0.08µs        ? ?/sec    1.03     19.5±0.07µs        ? ?/sec
ERC20/parse       1.00    377.7±2.96µs        ? ?/sec    1.01    383.4±4.90µs        ? ?/sec
SHA1/parse        1.00    175.2±4.21µs        ? ?/sec    1.03    180.1±3.50µs        ? ?/sec
Storage/parse     1.00     36.2±0.34µs        ? ?/sec    1.02     36.7±0.32µs        ? ?/sec
Transfer/parse    1.00     43.0±0.27µs        ? ?/sec    1.03     44.1±0.41µs        ? ?/sec

Lower Yul

Results
group             yul_lower_before                       yul_lower_changes
-----             ----------------                       -----------------
Baseline/lower    1.01   342.5±10.03µs        ? ?/sec    1.00    338.2±8.40µs        ? ?/sec
ERC20/lower       1.01  1631.3±32.88µs        ? ?/sec    1.00  1622.0±17.35µs        ? ?/sec
SHA1/lower        1.01   932.0±22.58µs        ? ?/sec    1.00   925.7±24.69µs        ? ?/sec
Storage/lower     1.00   422.6±10.18µs        ? ?/sec    1.01    425.0±8.18µs        ? ?/sec
Transfer/lower    1.00   454.5±11.12µs        ? ?/sec    1.00    454.6±8.94µs        ? ?/sec

@elle-j elle-j marked this pull request as ready for review January 15, 2026 06:21
permissions:
pull-requests: write
steps:
- name: Checkout Commit Before Changes
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Out of curiosity: Why does this need to be on top?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mainly for clarity to see at the top what's being checked out. Minor thing tho

@elle-j elle-j merged commit 2d85ca9 into main Jan 15, 2026
15 checks passed
@elle-j elle-j deleted the lj/fix-workflow branch January 15, 2026 08:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants