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

Lukezhuz/memacc #391

Closed
wants to merge 10 commits into from
Closed

Lukezhuz/memacc #391

wants to merge 10 commits into from

Conversation

9Tempest
Copy link

Mem Access Accelerator support in Polygeist.
Plan to work based on existing Polygeist framework in short term

@9Tempest 9Tempest closed this Mar 23, 2024
@9Tempest
Copy link
Author

Should push to my local fork. NVM

@wsmoses
Copy link
Member

wsmoses commented Mar 23, 2024

Oh I mean I think it would be cool to have it upstream, I just have been traveling the past few days and haven't had a chance tk look at.

@ivanradanov mind reviewing?

@wsmoses wsmoses reopened this Mar 23, 2024
@9Tempest
Copy link
Author

Oh I mean I think it would be cool to have it upstream, I just have been traveling the past few days and haven't had a chance tk look at.

@ivanradanov mind reviewing?

Hi William, thanks for your attention. This is a research artifact for mapping arbitrary indirect memory access to our memory accelerator, and we just start coding a day ago. I will do more work on that, and let’s see how we can upstream it. Also ideally we want to build this as a standalone dialect in the long term, I am using Polygeist because it has a nice template. The code needs more refactoring, and there’s no need review for now. I will let you know once it’s ready.

@MaizeHPC MaizeHPC closed this by deleting the head repository Mar 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants