Skip to content
This repository was archived by the owner on Feb 24, 2026. It is now read-only.

[TL] initial implement flashattention op in TL - #202

Merged
Lei Wang (LeiWang1999) merged 6 commits into
microsoft:mainfrom
tzj-fxz:main
Sep 30, 2024
Merged

[TL] initial implement flashattention op in TL#202
Lei Wang (LeiWang1999) merged 6 commits into
microsoft:mainfrom
tzj-fxz:main

Conversation

@tzj-fxz

Copy link
Copy Markdown
Contributor
  • add initial implementation of flashattention in ops/general_flashatten
  • wrap implementation in config class and op class
  • fix kernel name bug
  • modify tilelang test script
  • add ops flashattention test script

@LeiWang1999Lei Wang (LeiWang1999) left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for your contributions. Zhengju Tang (@tzj-fxz)
I've left some comments.

Comment threadbitblas/ops/general_flashatten/__init__.py Outdated
Comment threadbitblas/ops/operator.py Outdated
Comment threadbitblas/ops/operator.py Outdated
@LeiWang1999

Copy link
Copy Markdown
Contributor

thanks Zhengju Tang (@tzj-fxz) ,overall lgtm, should fix the test.

@tzj-fxz

Copy link
Copy Markdown
ContributorAuthor

thanks Zhengju Tang (@tzj-fxz) ,overall lgtm, should fix the test.

OK, now CI tests are passed in the newest commit.

@LeiWang1999
Lei Wang (LeiWang1999) merged commit 155a1f1 into microsoft:mainSep 30, 2024
@LeiWang1999

Copy link
Copy Markdown
Contributor

Merged, thanks Zhengju Tang (@tzj-fxz)

Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@tzj-fxz@LeiWang1999