Skip to content

Liveness analysis may not be adequate for SSA form #12

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

Open
dibyendumajumdar opened this issue Dec 22, 2024 · 1 comment
Open

Liveness analysis may not be adequate for SSA form #12

dibyendumajumdar opened this issue Dec 22, 2024 · 1 comment
Labels
fixed Fixed

Comments

@dibyendumajumdar
Copy link
Contributor

The discussion in the thread below indicates that the standard liveness calculation may not be handling Phi functions correctly.

Currently in our implementation, Phi's do not contribute uses (which is correct). But we do not do anything special with the values that feed into the Phi.

https://www.reddit.com/r/Compilers/comments/9qt31m/liveness_and_ssa/

@dibyendumajumdar
Copy link
Contributor Author

The new approach is based on formula described in
Computing Liveness Sets for SSA-Form Programs
Florian Brandner, Benoit Boissinot, Alain Darte, Benoît Dupont de Dinechin, Fabrice Rastello

The implementation is the unoptimized simple one.
However, we have a modification to ensure that if we see a block
which loops to itself and has Phi cycles, then the Phi is only added to
PhiDefs.

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

No branches or pull requests

1 participant