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

[Codegen] Implement Structs and PathOp #71

Open
edg-l opened this issue Jan 11, 2024 · 0 comments
Open

[Codegen] Implement Structs and PathOp #71

edg-l opened this issue Jan 11, 2024 · 0 comments
Labels
enhancement New feature or request
Milestone
0.01 Talc

Comments

@edg-l
Copy link
Member

edg-l commented Jan 11, 2024 •
edited
Loading

Structs need to be implemented.

Implementing structs also requires to properly implement PathOp, which is the operation used to access a field of a struct.

concrete/crates/concrete_codegen_mlir/src/codegen.rs

Line 110 in 771662e

ModuleDefItem::Record(_) => todo!(),

concrete/crates/concrete_codegen_mlir/src/codegen.rs

Line 508 in 771662e

fn compile_path_op<'c, 'op>(

concrete/crates/concrete_codegen_mlir/src/codegen.rs

Line 269 in 771662e

// todo: implement properly for structs, right now only really works for simple variables.

@edg-l edg-l added the enhancement New feature or request label Jan 11, 2024
@igaray igaray added this to the 0.0.1 Chalk milestone Jan 18, 2024
@edg-l edg-l mentioned this issue Jan 22, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
0.01 Talc
Development

No branches or pull requests
2 participants
@igaray @edg-l