Skip to content

x86 legalize: cardinality preservation is in fact not a requirement#5494

Open
qaco wants to merge 1 commit intoxdslproject:mainfrom
qaco:hugo/bug
Open

x86 legalize: cardinality preservation is in fact not a requirement#5494
qaco wants to merge 1 commit intoxdslproject:mainfrom
qaco:hugo/bug

Conversation

@qaco
Copy link
Collaborator

@qaco qaco commented Dec 7, 2025

No description provided.

@qaco qaco requested a review from superlopuh December 7, 2025 11:08
@qaco qaco self-assigned this Dec 7, 2025
@qaco qaco added the backend Compiler backend in xDSL label Dec 7, 2025
@codecov
Copy link

codecov bot commented Dec 7, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.95%. Comparing base (56e89d3) to head (33bc321).
⚠️ Report is 34 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5494      +/-   ##
==========================================
- Coverage   85.96%   85.95%   -0.01%     
==========================================
  Files         388      388              
  Lines       55262    55260       -2     
  Branches     6354     6354              
==========================================
- Hits        47505    47501       -4     
- Misses       6247     6248       +1     
- Partials     1510     1511       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@superlopuh
Copy link
Member

It would be nice to have a test with a minimal version of the code that wasn't correctly handled before and would be now, but more of a nit

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backend Compiler backend in xDSL

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants