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 generalization for DotLike operations #20

Closed
wants to merge 7 commits into from

Conversation

karupayun
Copy link
Member

@karupayun karupayun commented Dec 3, 2024

This PR allows other dot-like operations to be built on top of Triton instead of requiring ops to be DotOp exactly.

We are using this internally in order to support SparseDotOp (which has DotLike trait implemented).

- Add an `allocShape` field to denote the shape a memory descriptor when
it's allocated. The value will be propagated to all its descendants
created through `subview` ops.
- Make `encoding` and `memorySpace` fields required instead of optional.
- Implement the `getAlias` function for `#ttg.shared_memory` to shorten
its length in `.mlir` files
@karupayun karupayun requested a review from chsigg December 3, 2024 16:53
@karupayun karupayun force-pushed the refactor/generalize-dot-like-op branch from d6117aa to 51e2597 Compare December 3, 2024 17:13
scxiao and others added 5 commits December 3, 2024 12:05
This commit is to support atomic_rmw in the function
predicateOp to mask operations during scheduling.
PR triton-lang#5231 was authored before the `triton_gpu` -> `ttg` rename and CI is
currently broken.
…riton-lang#5314)

For other backends, `ptxas` may not be installed.

Signed-off-by: Anatoly Myachev <[email protected]>
This pulls in the AMDGPU backend support for the
gfx950 target.

We need to fix the rewrites in `Combine.td` given that
llvm/llvm-project#112700 adds
a new attribute for denorm mode for `arith.addf`.

---------

Co-authored-by: Lei Zhang <[email protected]>
Enable new arch target since backend support has been added.
Copy link
Member

@chsigg chsigg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, I just changed the description slightly.

@karupayun karupayun force-pushed the refactor/generalize-dot-like-op branch from 51e2597 to 98a9664 Compare December 4, 2024 15:22
@karupayun karupayun closed this Dec 4, 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.

7 participants