transformations: (csl-stencil-to-csl-wrapper) Translate func args to exportable memref.global

[n-io]

Top-level func args are no longer set up as block args in the newly created program module, but are instead translated to (exportable) memref.global ops.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.87%. Comparing base (4423038) to head (221d403).
Report is 14 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff            @@
##             main    #3051    +/-   ##
========================================
  Coverage   89.86%   89.87%            
========================================
  Files         415      416     +1     
  Lines       52078    52441   +363     
  Branches     8048     8115    +67     
========================================
+ Hits        46799    47130   +331     
- Misses       3984     4006    +22     
- Partials     1295     1305    +10     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[dk949]

I don't quite follow how this unrealized_conversion_cast gets passed to the stencil.load below. Is that done with the arg_op_mapping?

[n-io]

Any stencil.load should be resolved by --stencil-bufferize and not be there at this stage, and everything should be based on stencil.field<AxBxtensor<Cxf32>>. This in turn is translated to a memref in --csl-stencil-bufferize, which takes the element_type (tensor) and translates it to a memref. Maybe it should translate to memref<1x1xCxf32> instead of memref<Cxf32>, but this is all linked to DMP which has changes in the pipeline that allow it to operate on stencil.field, so it's best to wait for this to be merged.