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

Compile in-memory MLIR Module #514

Merged
merged 2 commits into from
Jul 21, 2023
Merged

Compile in-memory MLIR Module #514

merged 2 commits into from
Jul 21, 2023

Conversation

youben11
Copy link
Member

No description provided.

@cla-bot cla-bot bot added the cla-signed label Jul 20, 2023
@youben11 youben11 force-pushed the feat/compile-in-memory-module branch from 636c338 to 8ce5631 Compare July 20, 2023 13:57
Copy link
Contributor

@andidr andidr left a comment

Choose a reason for hiding this comment

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

I can't figure out where the Compilation Context is instantiated on the Python side in order to check that the associated MLIRContext lives long enough.

@youben11 youben11 force-pushed the feat/compile-in-memory-module branch from 8ce5631 to e66cd37 Compare July 20, 2023 15:01
@youben11
Copy link
Member Author

I can't figure out where the Compilation Context is instantiated on the Python side in order to check that the associated MLIRContext lives long enough.

Wasn't pushed sorry, here it is now https://github.com/zama-ai/concrete/pull/514/files#diff-6c99c7044f07fa639e664b590c593a676fbb5a73a1f52b7d267a950825a44038R161

@youben11 youben11 requested a review from andidr July 21, 2023 07:24
Copy link
Contributor

@umut-sahin umut-sahin left a comment

Choose a reason for hiding this comment

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

LGTM if CI is green!

Comment on lines +168 to +169
mlir_to_compile = mlir if isinstance(mlir, str) else str(mlir)
compilation_result = support.compile(mlir_to_compile, options)
Copy link
Contributor

Choose a reason for hiding this comment

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

We don't have support for in memory modules in jit?

@youben11 youben11 force-pushed the feat/compile-in-memory-module branch from 09c7ca5 to 2e7c9e2 Compare July 21, 2023 09:11
@youben11 youben11 force-pushed the feat/compile-in-memory-module branch from 2e7c9e2 to 882de25 Compare July 21, 2023 09:49
@youben11 youben11 merged commit 7e138bf into main Jul 21, 2023
@umut-sahin umut-sahin deleted the feat/compile-in-memory-module branch July 24, 2023 08:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants