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

Working BooleanLineVectorizer #838

Merged
merged 1 commit into from
Aug 5, 2024
Merged

Working BooleanLineVectorizer #838

merged 1 commit into from
Aug 5, 2024

Conversation

WoutLegiest
Copy link
Collaborator

  • Implementation of the BoolLineVectorizer, a pass in the CGGI dialect
  • Adding a test for the pass above
  • Expansion of the CGGI Dialect to support the representation of gates through attributes
  • Small refactoring of the TfheRustBool Dialect

@WoutLegiest WoutLegiest requested a review from asraa July 30, 2024 11:55
@WoutLegiest WoutLegiest mentioned this pull request Jul 30, 2024
Merged
asraa
asraa reviewed Jul 30, 2024
View reviewed changes
Copy link
Collaborator

@asraa asraa left a comment

Choose a reason for hiding this comment

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

Nice job!!!

lib/Dialect/CGGI/IR/CGGIOps.td Outdated Show resolved Hide resolved
lib/Dialect/CGGI/IR/CGGIOps.td Outdated Show resolved Hide resolved
lib/Dialect/CGGI/IR/CGGIOps.td Show resolved Hide resolved
lib/Dialect/CGGI/IR/CGGIAttributes.td Outdated Show resolved Hide resolved
lib/Dialect/CGGI/Transforms/Passes.td Outdated Show resolved Hide resolved
tests/cggi/boolean_line_vectorizer.mlir Outdated Show resolved Hide resolved
lib/Dialect/CGGI/Transforms/BooleanLineVectorizer.cpp Outdated Show resolved Hide resolved
lib/Dialect/CGGI/Transforms/BooleanLineVectorizer.cpp Show resolved Hide resolved
lib/Dialect/CGGI/Transforms/BooleanLineVectorizer.cpp Show resolved Hide resolved
lib/Dialect/CGGI/Transforms/BooleanLineVectorizer.cpp Outdated Show resolved Hide resolved
@WoutLegiest WoutLegiest added the pull_ready Indicates whether a PR is ready to pull. The copybara worker will import for internal testing label Aug 5, 2024
@copybara-service copybara-service bot merged commit ac2ef64 into google:main Aug 5, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@asraa asraa asraa left review comments

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.
2 participants
@WoutLegiest @asraa