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.

Sign up for GitHub

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

Jump to bottom

[TTNN] Merge consecutive ToLayoutOp #1551

Closed
sdjordjevicTT opened this issue Dec 10, 2024 · 0 comments · Fixed by #1552
Closed

[TTNN] Merge consecutive ToLayoutOp #1551

sdjordjevicTT opened this issue Dec 10, 2024 · 0 comments · Fixed by #1552
Assignees
@sdjordjevicTT
Labels
TTNN Dialect Issues related to TTNN dialect

Comments

@sdjordjevicTT
Copy link
Contributor

During the workaround pass, we insert a bunch of ToLayoutOps to apply workarounds. We need to add a canonicalizer for ToLayoutOp to merge two consecutive ToLayoutOps into one.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sdjordjevicTT sdjordjevicTT

Labels
TTNN Dialect Issues related to TTNN dialect
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[TTNN] Merging consecutive ttnn ToLayoutOps tenstorrent/tt-mlir
1 participant
@sdjordjevicTT