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

Adding Loop support to arith-to-cggi pass #1340

Merged
merged 1 commit into from
Jan 31, 2025
Merged

Adding Loop support to arith-to-cggi pass #1340

merged 1 commit into from
Jan 31, 2025
+173 −35

Conversation

WoutLegiest
Copy link
Collaborator

Lower the MNIST Tosa model to CGGI and keeping parallel loops
Redefinition of the memref conversion:

  • Memref can be kept in the arith world if it only stores constants
  • Now if the same allocOp is used to store both constants and lwe elements: bring to LWE world and add createTrivalOps
  • If AllocOp changes: add attribute to the affected operations. This attribute is used to convert the store and load ops

@j2kun j2kun self-requested a review January 30, 2025 18:20
@WoutLegiest WoutLegiest force-pushed the mnist_hl branch 2 times, most recently from 55f0c6c to 56fc5e1 Compare January 31, 2025 00:03
@WoutLegiest WoutLegiest requested a review from j2kun January 31, 2025 00:11
@WoutLegiest WoutLegiest force-pushed the mnist_hl branch 2 times, most recently from 05238fa to 137244f Compare January 31, 2025 01:19
@j2kun j2kun added the pull_ready Indicates whether a PR is ready to pull. The copybara worker will import for internal testing label Jan 31, 2025
@copybara-service copybara-service bot merged commit b063060 into google:main Jan 31, 2025
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@j2kun j2kun

Assignees
No one assigned
Labels
pull_ready Indicates whether a PR is ready to pull. The copybara worker will import for internal testing
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@WoutLegiest @j2kun