compiler: catch corner case read after write - #2909
Conversation
Sorry, something went wrong.
|
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
Sorry, something went wrong.
Codecov Report✅ All modified and coverable lines are covered by tests. @@ Coverage Diff @@
## main #2909 +/- ##
=======================================
Coverage 83.38% 83.39%
=======================================
Files 248 248
Lines 51685 51699 +14
Branches 4460 4460
=======================================
+ Hits 43100 43114 +14
+ Misses 7831 7830 -1
- Partials 754 755 +1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry.
|
Sorry, something went wrong.
|
|
||
| op = Operator(eqns) | ||
|
|
||
| assert_structure( |
There was a problem hiding this comment.
Maybe a brief comment explaining why fissioning in x is expected would be useful in the future?
Sorry, something went wrong.
| continue | ||
|
|
||
| distance = dependence.distance | ||
|
|
There was a problem hiding this comment.
leftover, or apply same style change to the other sibling methods
Sorry, something went wrong.
There was a problem hiding this comment.
This actually applies the same style as d_anti below that has the blank line but can remove blank line from both too if prefered
Sorry, something went wrong.
| assert all(trees[1].root.dim is tree.root.dim for tree in trees[1:]) | ||
|
|
||
|
|
||
| def test_scheduling_no_deriv(): |
There was a problem hiding this comment.
maybe mention "interpolation" in the name, or it may go underestimated
Sorry, something went wrong.
tentative, trying to see what can do for these cases