-
Notifications
You must be signed in to change notification settings - Fork 85
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: (csl_stencil) Add bufferization pass #3004
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #3004 +/- ##
==========================================
+ Coverage 89.83% 89.85% +0.02%
==========================================
Files 411 412 +1
Lines 51674 51816 +142
Branches 8034 8064 +30
==========================================
+ Hits 46419 46557 +138
+ Misses 3970 3968 -2
- Partials 1285 1291 +6 ☔ View full report in Codecov by Sentry. |
Can you please assign yourself and add the appropriate labels? |
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'm not getting all semantics used so just reviewed the code itself!
Change
stencil.field
to memrefs, change all tensors to memrefs, and perform a 'partial bufferization' by insertingbufferization.to_tensor
andbufferization.to_memref
ops where needed to that other dialects can still operate on tensors.Todo:
restrict
,writable
, andread_only
of thebufferization
opsEdit: I think I re-used a branch name of a merged branch, ignore some of the commits