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

First-Class Support for Callstring-derived Contexts #1379

Closed
michael-schwarz opened this issue Mar 2, 2024 · 1 comment · Fixed by #1427
Closed

First-Class Support for Callstring-derived Contexts #1379

michael-schwarz opened this issue Mar 2, 2024 · 1 comment · Fixed by #1427

Comments

@michael-schwarz
Copy link
Member

After #1340 has landed, we might want to generalize our constraint system to support Callstring-like contexts in a more natural manner.

I think the only change would be to change

  val context : fundec -> D.t -> C.t

to

  val context : fundec -> fundec -> C.t -> D.t -> C.t

where the additonal fundec would be the caller fundec.

That should be sufficiently general to allow call-string style contexts without keeping the callstring in a flat lattice inside the domain.

@sim642
Copy link
Member

sim642 commented Mar 2, 2024

I haven't checked if this is possible, but maybe we could even add ctx to the arguments of context. That gives access to the caller node (and thus its fundec). But also queries, which maybe could also be beneficial to have access to?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants