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

Implement address generator synthesis #27

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Conversation

ethanuppal
Copy link
Owner

@ethanuppal ethanuppal commented Oct 20, 2024

Implements #25.

@ethanuppal ethanuppal self-assigned this Oct 20, 2024
@ethanuppal ethanuppal added enhancement New feature or request roadblock Must be completed first labels Oct 20, 2024
@ethanuppal
Copy link
Owner Author

ethanuppal commented Oct 21, 2024

Currently this is very naive and incomplete. Here's a nonexhaustive list of things to do:

  • Have TimingAnalysis do a kind of "polyhedral analysis" and determine latency of loops with non constant integer bounds.
  • Instead of inserting delay at the start, see if you can do something smarter that's more susceptible to par-CFG merges.
  • And of course, clean up the code.

@ethanuppal
Copy link
Owner Author

Would it be possible to just separate out the parts of the program that compute the addresses explicitly?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request roadblock Must be completed first
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fix address generation for writes + generalize to arbitrary-dimension arrays
1 participant