Skip to content

ci: cross-OS wall-clock benchmark of the pool dispatch modes - #648

Closed
abinavpp wants to merge 14 commits into
app-rework-std-comsfrom
app-pool-mode-bench
Closed

ci: cross-OS wall-clock benchmark of the pool dispatch modes#648
abinavpp wants to merge 14 commits into
app-rework-std-comsfrom
app-pool-mode-bench

Conversation

@abinavpp

Copy link
Copy Markdown
Contributor

Opt-in CI benchmark (label ci:mode-bench) comparing in-process threads vs the SOLX_SUBPROCESS worker pool on all slang-tests platforms. Numbers feed the threads-by-default discussion on #587.

…annels

The exit()-based stack error handler and the per-unit LLVM option
parsing are process-global, making process-per-unit isolation a
correctness requirement rather than a performance choice. Stack-too-deep
now arrives as a per-context diagnostic captured around emission, and
per-unit codegen parameters ride the module as flags. Needs the paired
solx-llvm branch (per-module flag reading, the diagnostic kind and its
C API accessor).
@abinavpp abinavpp added the ci:mode-bench Run the pool-mode wall-clock benchmark label Aug 14, 2026
@abinavpp
abinavpp force-pushed the app-pool-mode-bench branch from 4a0032d to d3221b1 Compare August 14, 2026 10:00
@abinavpp
abinavpp force-pushed the app-pool-mode-bench branch from 58401fc to a16a794 Compare August 14, 2026 16:09
@abinavpp
abinavpp force-pushed the app-pool-mode-bench branch from 12ca10a to 57d06cb Compare August 15, 2026 15:42
@github-actions

Copy link
Copy Markdown

Coverage Summary

Crate Line Coverage Function Coverage
solx 🟡 72.6% 🔴 40.0%
solx-benchmark-converter 🔴 0.0% 🔴 0.0%
solx-codegen-evm 🟡 61.4% 🟡 62.8%
solx-compiler-downloader 🔴 0.0% 🔴 0.0%
solx-core 🔴 47.5% 🟡 59.1%
solx-dev 🔴 2.4% 🔴 3.0%
solx-evm-assembly 🟡 51.0% 🟡 54.4%
solx-solc-test-adapter 🔴 1.7% 🔴 2.1%
solx-standard-json 🟡 53.8% 🟡 57.5%
solx-tester 🔴 38.1% 🔴 35.9%
solx-utils 🟡 61.6% 🟡 59.4%
solx-yul 🟡 54.0% 🟡 60.4%
Total 🔴 36.6% 🔴 36.6%

Codecov Report | HTML Report | Workflow Run

@abinavpp
abinavpp force-pushed the app-rework-std-coms branch from 59e16a8 to dffd7e3 Compare August 19, 2026 06:38
@abinavpp

Copy link
Copy Markdown
Contributor Author

Folded into #587 (the benchmark harness, workflow, and report now live there).

@abinavpp abinavpp closed this Aug 19, 2026
@hedgar2017
hedgar2017 deleted the app-pool-mode-bench branch August 19, 2026 13:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci:mode-bench Run the pool-mode wall-clock benchmark

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant