Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add an Instance resolve for the closure function itself #85

Open
celinval opened this issue Jul 11, 2024 · 1 comment
Open

Add an Instance resolve for the closure function itself #85

celinval opened this issue Jul 11, 2024 · 1 comment

Comments

@celinval
Copy link
Contributor

StableMIR doesn't provide an easy to way to retrieve the body of a closure. Instance::resolve_closure will return a shim.

The other possibility would be to allow users to convert ClosureDef to FnDef, but we should make sure that doesn't break any usage of FnDef. @oli-obk, any preference here?

@oli-obk
Copy link
Contributor

oli-obk commented Jul 15, 2024

If there's no issues with using FnDef, that seems best. From the current set of queries we expose on FnDef, I don't think we'll have any problems

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants