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 SparseConstantPropagation to the data flow solver demo #34

Open
j2kun opened this issue Nov 29, 2023 · 0 comments
Open

Add SparseConstantPropagation to the data flow solver demo #34

j2kun opened this issue Nov 29, 2023 · 0 comments

Comments

@j2kun
Copy link
Owner

j2kun commented Nov 29, 2023

See https://discourse.llvm.org/t/mlir-dead-code-analysis/67568/8

I tried out the data flow analysis framework, and also realized the DeadCodeAnalysis was required for pretty much every analysis I wanted to do. So I’m sympathetic to making this easier to get started with, but I’m also curious what making it a “built-in” feature would entail. Maybe rather than built-in in the sense of “always included in DataFlowSolver” is too strong, but we could provide some helper that includes some sensible defaults (e.g. DeadCodeAnalysis and SparseConstantPropagation), which users could extend.

Otherwise it won't properly handle control flow, which I didn't test enough in my tutorial, but ran into when I was working on this in HEIR.

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

No branches or pull requests

1 participant