-
Notifications
You must be signed in to change notification settings - Fork 229
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
compiler: Fix subdim argument mismatch #2019
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2019 +/- ##
==========================================
- Coverage 87.78% 87.78% -0.01%
==========================================
Files 214 214
Lines 37017 37039 +22
Branches 5573 5577 +4
==========================================
+ Hits 32496 32515 +19
- Misses 4000 4003 +3
Partials 521 521
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
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.
basically no objections here, just minor comments
f79c01a
to
af3d517
Compare
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.
LGTM, thanks
e96aec7
to
8385cf1
Compare
8385cf1
to
651e12c
Compare
""" | ||
Generate C code. |
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.
I would keep this
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.
On top of #2018
Fix a small corner case where you can end with something like
[nt, frozenset{nt, nt+1, ...}]
needing to be reduced/unique'ed with subdimensions and apply overrides.