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
testing: Merge and simplify convert-stencil-to-ll-mlir filechecks. #1046
Conversation
1f5b6fb
to
077513c
Compare
Codecov ReportPatch coverage has no change and project coverage change:
Additional details and impacted files@@ Coverage Diff @@
## emilien/stencil-filecheck-polish #1046 +/- ##
====================================================================
- Coverage 86.95% 86.92% -0.04%
====================================================================
Files 126 126
Lines 19434 19434
Branches 2948 2948
====================================================================
- Hits 16899 16893 -6
- Misses 2036 2041 +5
- Partials 499 500 +1 ☔ View full report in Codecov by Sentry. |
I'll add some examples to get the coverage at least back to before the PR (for the stencil code; don't know about those parser internal corner-cases!) |
077513c
to
ff6249a
Compare
Stacked on #1047 because part of the coverage loss is due to an issue with the shape inference's filecheck. |
Some were just removed, e.g. separate load/cast/access tests, which are just tested in every example.
Thos can be thought of as reconciled unrealized casts.
0ad9e64
to
7a03f56
Compare
At time of writing the Codecov comment takes time to update but I checked codecov report for the loss of coverage and it's alright now (https://app.codecov.io/gh/xdslproject/xdsl/pull/1046) |
Some were just removed, e.g. separate load/cast/access tests, which are just tested in every example.
More importantly, this switches all examples to lowering-only of shape infered examples, to separately test shape-inference and conversion from now on.