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

Adapting the TfheRustBool dialect to pass the packed api + update cggi to tfhe rust bool conversion pass #842

Merged
merged 1 commit into from
Oct 12, 2024

Conversation

WoutLegiest
Copy link
Collaborator

Follow up work of #838

@ai-mannamalai
Copy link
Contributor

@WoutLegiest - can you consider adding the CMake files for the new dialect and the transform pass ? It should be straight forward

lib/Dialect/CGGI/Transforms/Passes.td Outdated Show resolved Hide resolved
lib/Dialect/CGGI/Transforms/BooleanLineVectorizer.cpp Outdated Show resolved Hide resolved
lib/Dialect/CGGI/Transforms/BooleanLineVectorizer.cpp Outdated Show resolved Hide resolved
lib/Dialect/CGGI/Transforms/BooleanLineVectorizer.cpp Outdated Show resolved Hide resolved
tests/cggi/boolean_line_vectorizer.mlir 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 6, 2024
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.

Thank you!

lib/Dialect/CGGI/Transforms/BooleanVectorizer.cpp Outdated Show resolved Hide resolved
lib/Dialect/TfheRustBool/IR/TfheRustBoolAttributes.td Outdated Show resolved Hide resolved
lib/Dialect/TfheRustBool/IR/TfheRustBoolAttributes.td Outdated Show resolved Hide resolved
@ai-mannamalai
Copy link
Contributor

adding a PR WoutLegiest#1 into your branch to enable CMake build in order

@WoutLegiest WoutLegiest force-pushed the dev-conv2 branch 3 times, most recently from 698dcfb to fb44d32 Compare August 15, 2024 15:53
@WoutLegiest WoutLegiest requested a review from j2kun August 16, 2024 08:41
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.

Thank you! Just a couple minor nits now, but otherwise LGTM!

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 Outdated Show resolved Hide resolved
lib/Dialect/CGGI/IR/CGGIEnums.td Show resolved Hide resolved
lib/Dialect/CGGI/Transforms/Passes.td Show resolved Hide resolved
lib/Dialect/CGGI/Transforms/Passes.td Outdated Show resolved Hide resolved
lib/Dialect/TfheRustBool/IR/TfheRustBoolOps.td Outdated Show resolved Hide resolved
lib/Dialect/TfheRustBool/IR/TfheRustBoolOps.td Outdated Show resolved Hide resolved
lib/Dialect/CGGI/Transforms/BooleanVectorizer.cpp Outdated Show resolved Hide resolved
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.

Thank you!

@asraa
Copy link
Collaborator

asraa commented Sep 5, 2024

Sorry about this! I think this just needed a rebase before merging

@asraa
Copy link
Collaborator

asraa commented Oct 9, 2024

Going to rebase this one myself!

@asraa asraa force-pushed the dev-conv2 branch 3 times, most recently from 9cf3a93 to b7d5aab Compare October 12, 2024 00:01
…s update

Adding the (common) enum attribute to model Boolean gates

Co-authored-by: Wouter Legiest <[email protected]>
Co-authored-by: Asra Ali <[email protected]>
@copybara-service copybara-service bot merged commit c220b57 into google:main Oct 12, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pull_ready Indicates whether a PR is ready to pull. The copybara worker will import for internal testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants