Skip to content

⚡ Expose assertion-free MLIR toolchains - #255

Open
burgholzer wants to merge 3 commits into
mainfrom
codex/optimized-release-toolchain
Open

⚡ Expose assertion-free MLIR toolchains#255
burgholzer wants to merge 3 commits into
mainfrom
codex/optimized-release-toolchain

Conversation

@burgholzer

@burgholzer burgholzer commented Sep 8, 2026

Copy link
Copy Markdown
Member

🤖 AI text below 🤖

Adds assertions: false to select the optimized Release archives from munich-quantum-software/portable-mlir-toolchain#94. Existing callers and legacy manifest entries retain assertions; unavailable variants fail explicitly. Debug and assertion-free toolchains use distinct cache directories.

The manifest generator recognizes both archive variants. Standalone installers expose -a OFF (Bash) and -no_assertions (PowerShell).

Validation: 10 focused manifest, selection, fallback, and input-validation tests pass; TypeScript checking, regenerated action bundles, Bash syntax checking, and all pre-commit checks pass. PowerShell execution and downloads of the new archives await hosted validation and publication of the toolchain artifacts.

Both SDK variants contain native archives. Consumers may enable LTO for their own code without an exact compiler-version match; normal ABI and runtime requirements remain. Linux release SDKs supply BOLT tooling. README lint passes after this contract update.

Depends on munich-quantum-software/portable-mlir-toolchain#94 and publication of its assertion-free archives.

Consumers: munich-quantum-toolkit/workflows#464 and munich-quantum-toolkit/core#2476.

Keep assertion-enabled defaults and isolate SDK cache variants.

Assisted-by: GPT-6 via Codex
@codecov

codecov Bot commented Sep 8, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 86.36364% with 3 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/utils/download.ts 66.66% 2 Missing ⚠️
src/index.ts 85.71% 1 Missing ⚠️

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #255      +/-   ##
==========================================
- Coverage   88.71%   88.44%   -0.28%     
==========================================
  Files           7        7              
  Lines         390      398       +8     
  Branches       94      100       +6     
==========================================
+ Hits          346      352       +6     
- Misses         44       46       +2     
Flag Coverage Δ
integration-macos-26 81.65% <86.36%> (-0.14%) ⬇️
integration-macos-26-intel 81.40% <86.36%> (-0.14%) ⬇️
integration-ubuntu-24.04 81.15% <86.36%> (-0.13%) ⬇️
integration-ubuntu-24.04-arm 81.40% <86.36%> (-0.14%) ⬇️
integration-windows-11-arm 87.18% <86.36%> (-0.25%) ⬇️
integration-windows-2025 86.93% <86.36%> (-0.25%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
src/utils/manifest.ts 93.65% <100.00%> (+0.06%) ⬆️
src/index.ts 80.34% <85.71%> (-0.19%) ⬇️
src/utils/download.ts 93.47% <66.66%> (-1.98%) ⬇️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Describe the matched Linux images and Xcode needed for LTO archives.

Assisted-by: GPT-6 via Codex
@burgholzer
burgholzer force-pushed the codex/optimized-release-toolchain branch from 995b7a1 to dae31ad Compare September 8, 2026 21:17
Remove exact compiler and image matching requirements for native archives.

Assisted-by: GPT-6
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

continuous integration CI-related changes enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant