Skip to content
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

Backward Conversion #27

Merged
merged 52 commits into from
Sep 11, 2023
Merged

Backward Conversion #27

merged 52 commits into from
Sep 11, 2023

Conversation

Berke-Ates
Copy link
Collaborator

@Berke-Ates Berke-Ates commented Jul 4, 2023

This PR adds a partial backwards conversion pass (or a lowering pass) to the SDFG dialect, which can be called using sdfg-opt --lower-sdfg. The tests folder structure has been adapted and additional tests have been added. Here is an overview of the operations and their implementation status in the converter:

  • sdfg.sdfg
  • sdfg.nested_sdfg
  • sdfg.state
  • sdfg.edge (with assignments and conditionals)
  • sdfg.alloc (only array type with symbolic and constant sizes)
  • sdfg.load (with symbolic and constant indices)
  • sdfg.store (with symbolic and constant indices)
  • sdfg.alloc_symbol
  • sdfg.sym
  • sdfg.tasklet
  • sdfg.return
  • sdfg.map (with symbolic and constant bounds and step)
  • sdfg.consume
  • sdfg.subview
  • sdfg.view_cast
  • Stream Type

All users of symbolic expressions (such as maps, edges, loads, stores, etc.) use a symbolic parser, which implements a lexer, parser and an internal AST to convert the symbolic expression to builtin MLIR operations.

@Berke-Ates Berke-Ates requested a review from acalotoiu July 5, 2023 07:09
@acalotoiu
Copy link
Contributor

Hi! Could you please follow the guide here: https://github.com/spcl/dace/blob/master/CONTRIBUTING.md
Especially with respect to documentation and also documentation of tests.

Thanks!

Copy link
Contributor

@acalotoiu acalotoiu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(See previous comment)

@Berke-Ates Berke-Ates merged commit 5067939 into main Sep 11, 2023
2 checks passed
@Berke-Ates Berke-Ates deleted the backward-conversion branch September 11, 2023 18:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants