Skip to content

Added SEAL library and FHE mode - #41

Open
Northrim wants to merge 27 commits into
masterfrom
fhe
Open

Northrim wants to merge 27 commits into
masterfrom
fhe

Conversation

@Northrim

Copy link
Copy Markdown
Collaborator

No description provided.

@Northrim
Northrim requested review from alex-ozdemir and edwjchen and removed request for alex-ozdemir January 10, 2022 21:29

@alex-ozdemir alex-ozdemir left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good so far, William. I have a few suggestions within.

Comment thread .gitmodules Resolved
Comment thread src/front/zokrates/mod.rs Resolved
Comment thread examples/circ.rs Resolved
@Northrim
Northrim requested a review from alex-ozdemir March 25, 2022 05:18

@alex-ozdemir alex-ozdemir left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

A few questions...

Comment thread .vscode/settings.json Resolved
Comment thread src/target/fhe/utils.rs Resolved
Comment thread driver.py Resolved
Comment thread scripts/build_fhe_c_test.zsh Resolved
Comment thread scripts/build_fhe_zokrates_test.zsh Resolved
Comment thread scripts/seal_tests/c_test_seal.py Resolved
Comment thread scripts/seal_tests/c_test_seal.py Resolved
Comment thread src/front/c/mod.rs Resolved
Comment thread src/target/fhe/trans.rs Resolved
Comment thread src/target/fhe/trans.rs Resolved
Comment thread src/target/fhe/trans.rs Resolved

@edwjchen edwjchen left a comment •

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Left a few small comments!

@alex-ozdemir when you have a minute, could you take a look at the Map IR lowering?

Comment thread scripts/build_fhe_zokrates_test.zsh Resolved
Comment thread .vscode/settings.json Resolved
Comment thread scripts/seal_tests/util.py Resolved
Comment thread src/target/fhe/trans.rs Resolved
Comment thread src/target/fhe/utils.rs Resolved

@alex-ozdemir alex-ozdemir left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I left a nit and two questions.

Comment thread src/target/fhe/trans.rs Resolved
Comment thread src/target/fhe/trans.rs Resolved
@edwjchen
edwjchen self-requested a review April 27, 2022 05:28

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants