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

Rewrite Index Tree Dialect #79

Draft
wants to merge 77 commits into
base: master
Choose a base branch
from
Draft

Rewrite Index Tree Dialect #79

wants to merge 77 commits into from

Conversation

AK2000
Copy link
Collaborator

@AK2000 AK2000 commented Jan 15, 2025

Over the past year, we undertook nearly a complete rewriting of the index tree dialect in order enable future analysis and transformation. These changes included:

  • Reversing the structure of the index "tree" so nodes higher in the conceptual tree also occur higher in the IR, and "leaf" nodes in the tree occur at the bottom. Not only does this more naturally match the intution of the tree, it also enables properties of the tree to be enforced by the IR. Specifically, each IntexNodeOp only accepts a single "parent" index

AK2000 added 30 commits October 14, 2024 17:47
AK2000 and others added 30 commits November 5, 2024 16:24
Decouple semiring attribute from domain inference
…runner. Also updated README.md to reflect this change
…ger type (i32,i64) instead

of index, to allow the frontends choose how they want to represent them. #69
Also, fixed test to correctly reflect its name
Fix masking in index tree rewrite
* [WIP] Lowering dense domain to parallel loop

* [WIP] fixing errors when building with parallel op

* [WIP] Limited parallelism working on spmm CSRxDense case

* Trying to fix indexing into tensor slices

* Removing parallelism for sparse output. Fixing problems with dense parallelism

* Fixing issue with COO and parallel loops
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