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

Add dominator Tree #1116

Closed
1 task done
kevaundray opened this issue Apr 7, 2023 · 1 comment · Fixed by #1278
Closed
1 task done

Add dominator Tree #1116

kevaundray opened this issue Apr 7, 2023 · 1 comment · Fixed by #1278
Assignees
Labels
enhancement New feature or request refactor ssa

Comments

@kevaundray
Copy link
Contributor

Problem

Dominator trees store the dominance relation between basic blocks in a function.

A dominance tree allows us to quickly compute:

  • The dominance frontier
  • Immediate dominators for a node

The dominator tree is very useful in figuring out which nodes are not dominated by the entry block and for example, can be used for reachability analysis.

Proposed solution

Implement a dominator tree in a modular way with comments and tests

Alternatives considered

No response

Additional context

No response

Submission Checklist

  • Once I hit submit, I will assign this issue to the Project Board with the appropriate tags.
@kevaundray kevaundray added the enhancement New feature or request label Apr 7, 2023
@kevaundray kevaundray added this to Noir Apr 7, 2023
@github-project-automation github-project-automation bot moved this to 📋 Backlog in Noir Apr 7, 2023
@guipublic
Copy link
Contributor

For the record, we already have the dominator tree implemented in the BasicBlock structure where blocks reference their direct dominator.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request refactor ssa
Projects
Archived in project
3 participants