Skip to content

dialects: (llvm) add FCmpOp#5706

Open
sueszli wants to merge 6 commits intoxdslproject:mainfrom
sueszli:llvm.fcmp
Open

dialects: (llvm) add FCmpOp#5706
sueszli wants to merge 6 commits intoxdslproject:mainfrom
sueszli:llvm.fcmp

Conversation

@sueszli
Copy link
Contributor

@sueszli sueszli commented Mar 7, 2026

No description provided.

@codecov
Copy link

codecov bot commented Mar 7, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.35%. Comparing base (61608e7) to head (5f79bf8).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5706      +/-   ##
==========================================
+ Coverage   86.34%   86.35%   +0.01%     
==========================================
  Files         406      406              
  Lines       57753    57800      +47     
  Branches     6709     6711       +2     
==========================================
+ Hits        49869    49916      +47     
  Misses       6325     6325              
  Partials     1559     1559              

☔ 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.

@sueszli sueszli marked this pull request as draft March 7, 2026 10:51
@sueszli
Copy link
Contributor Author

sueszli commented Mar 7, 2026

Needs some more code quality improvements. Created PR too early.

@sueszli sueszli marked this pull request as ready for review March 7, 2026 11:21
@sueszli
Copy link
Contributor Author

sueszli commented Mar 7, 2026

Done


@property
def int_value(self) -> int:
return list(type(self)).index(self)
Copy link
Member

Choose a reason for hiding this comment

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

why remove the old mechanism with a dict?

Copy link
Member

Choose a reason for hiding this comment

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

I'd rather duplicate the code TBH

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