-
Notifications
You must be signed in to change notification settings - Fork 49
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
feat[next][dace]: DaCe support for temporaries #1351
Merged
Merged
Changes from all commits
Commits
Show all changes
32 commits
Select commit
Hold shift + click to select a range
3c38362
[dace] Lift support with ITIR temporaries
edopao 8fd0fda
[dace] Replace unicode symbols from SSA ITIR pass
edopao 1020248
Merge branch 'GridTools:main' into dace-temporaries
edopao 6a46682
[dace] Disable simplify for SDFG with temporaries
edopao 7b251f8
[dace] Bugfix for array offset
edopao e2ff6f6
[dace] Fix for array offset in simplify pass
edopao 2380efb
Revert "[dace] Bugfix for array offset"
edopao 5f10d3d
[dace] Fix for ITIR temporary pass
edopao 32052b6
[dace] Fix type-check error
edopao 842a340
Merge remote-tracking branch 'origin/main' into dace-temporaries
edopao e65cbb2
[dace] Fix code-stile errors
edopao 326748a
[dace] Add comments
edopao fec7ae4
[dace] Improve string to enum conversion
edopao fa7482d
Merge branch 'GridTools:main' into dace-temporaries
edopao 81f3f74
[dace] Add kwarg for lift_mode
edopao 240de21
[dace] Force temporaries for LIFT
edopao 22041e4
Merge remote-tracking branch 'origin/main' into dace-temporaries
edopao 112a594
[dace] Minor edit
edopao b8a5db7
Merge remote-tracking branch 'origin/main' into dace-temporaries
edopao da795ef
Merge remote-tracking branch 'origin/main' into dace-temporaries
edopao c387b38
[dace] Re-added simplify step
edopao 6c5668f
[dace] Conform with error message guidelines
edopao 4187ee1
Merge remote-tracking branch 'origin/main' into dace-temporaries
edopao 67fc2cf
[dace] Enable lift_mode arg
edopao 3e80907
[dace] Fix formatting
edopao 55250db
Merge remote-tracking branch 'origin/main' into dace-temporaries
edopao 8e203e8
Minor edit
edopao 15a693f
Fix error (lift_mode was ignored)
edopao 1a7da6a
Minor edit
edopao 3529244
[dace] Remove pass for SSA identifiers
edopao 9aa3629
[dace] Review comments
edopao 8b74107
[dace] Remove tasklet for symbolic expressions
edopao File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
You spend ~70 lines on filling
tmp_symbols
and does not explains what the loop does or the variable represents.But for the small loop above you write a description, this seems a bit strange.