Skip to content

fix(evm): describe the spill region with module flags - #690

Open
abinavpp wants to merge 2 commits into
main-solcfrom
app-spill-region-flags-solc
Open

fix(evm): describe the spill region with module flags#690
abinavpp wants to merge 2 commits into
main-solcfrom
app-spill-region-flags-solc

Conversation

@abinavpp

@abinavpp abinavpp commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

Copy of #653 for main-solc.

The region is named on the module rather than on the command line, so
the backend has one interface and the pipeline stops depending on
process-global options.
The options the region used to travel on are gone from the pinned
revision onwards.
@github-actions

github-actions Bot commented Sep 4, 2026

Copy link
Copy Markdown

Coverage Summary

Crate Line Coverage Function Coverage
solx 🟡 72.7% 🔴 40.0%
solx-benchmark-converter 🔴 0.0% 🔴 0.0%
solx-codegen-evm 🟡 61.5% 🟡 63.0%
solx-compiler-downloader 🔴 0.0% 🔴 0.0%
solx-core 🟡 50.3% 🟡 62.8%
solx-dev 🔴 2.4% 🔴 3.0%
solx-evm-assembly 🟡 50.9% 🟡 54.5%
solx-solc-test-adapter 🔴 1.7% 🔴 2.1%
solx-standard-json 🟡 53.9% 🟡 57.5%
solx-tester 🔴 37.9% 🔴 35.7%
solx-utils 🟡 61.6% 🟡 60.0%
solx-yul 🟡 54.0% 🟡 60.4%
Total 🔴 36.9% 🔴 36.9%

Codecov Report | HTML Report | Workflow Run

@hedgar2017 hedgar2017 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please see #653.

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