-
Notifications
You must be signed in to change notification settings - Fork 72
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
transformations: (scf-to-cf) add lowering for index_switch #3179
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #3179 +/- ##
==========================================
+ Coverage 90.03% 90.04% +0.01%
==========================================
Files 430 432 +2
Lines 54183 54483 +300
Branches 8392 8453 +61
==========================================
+ Hits 48784 49060 +276
- Misses 4043 4056 +13
- Partials 1356 1367 +11 ☔ View full report in Codecov by Sentry. |
case_value, | ||
default_block, | ||
(), | ||
DenseIntOrFPElementsAttr.vector_from_list(case_values, i32), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can this not be the op.cases
above? Does the type change?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, op.cases
is a DenseArrayBase
but this is a DenseIntOrFPElementsAttr
and these appear to not be the same
Co-authored-by: Sasha Lopoukhine <superlopuh@gmail.com>
Again this is mainly copied from the implementation in mlir