Skip to content

[Gluon] Rename async_copy_... -> async_{load,store}#10083

Merged
peterbell10 merged 2 commits into
mainfrom
pb/gluon-tma
Apr 20, 2026
Merged

[Gluon] Rename async_copy_... -> async_{load,store}#10083
peterbell10 merged 2 commits into
mainfrom
pb/gluon-tma

Conversation

@peterbell10
Copy link
Copy Markdown
Contributor

@peterbell10 peterbell10 commented Apr 20, 2026

This renames both the ampere style async_copy and tma functions. The original names are pretty much copied directly from ptx and while they are very descriptive, are also rather verbose.

IMO these shorter names have a better glance-value and better conveys the intent of the programmer.

I've also kept the original names as aliases to preserve backward-compatibility.

The async copy names are pretty much copied directly from ptx. While
they are very descriptive, it's also rather verbose.

IMO this has a better glance-value and conveys the actual intent.
Copy link
Copy Markdown
Contributor

@lezcano lezcano left a comment

Choose a reason for hiding this comment

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

Amazing!

@peterbell10 peterbell10 merged commit ec35f3d into main Apr 20, 2026
9 checks passed
@peterbell10 peterbell10 deleted the pb/gluon-tma branch April 20, 2026 15:22
bingyizh233 pushed a commit to bingyizh233/triton that referenced this pull request Apr 20, 2026
This renames both the ampere style async_copy and tma functions. The
original names are pretty much copied directly from ptx and while they
are very descriptive, are also rather verbose.

IMO these shorter names have a better glance-value and better conveys
the intent of the programmer.

I've also kept the original names as aliases to preserve
backward-compatibility.
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.

2 participants