Skip to content

[triton-ext] Insert return Value into args for TritonOpBuilder#9864

Merged
ThomasRaoux merged 1 commit into
mainfrom
CRobeck-patch-1
Mar 26, 2026
Merged

[triton-ext] Insert return Value into args for TritonOpBuilder#9864
ThomasRaoux merged 1 commit into
mainfrom
CRobeck-patch-1

Conversation

@CRobeck
Copy link
Copy Markdown
Contributor

@CRobeck CRobeck commented Mar 26, 2026

#9748 introduced some changes to the custom ops API and dropped the return value. But the plugin convention expects operands[0] to be a writable result slot. This adds that return value for the added op back.

@CRobeck
Copy link
Copy Markdown
Contributor Author

CRobeck commented Mar 26, 2026

cc: @abrown

@abrown
Copy link
Copy Markdown
Contributor

abrown commented Mar 26, 2026

We talked about this on Slack for a bit: for whoever reviews this, I think this is a decent solution to add back what got dropped in #9748 (I missed this returned value in one of the many rebase/merges to keep that PR up to date).

@CRobeck CRobeck marked this pull request as ready for review March 26, 2026 20:09
@CRobeck CRobeck requested a review from ptillet as a code owner March 26, 2026 20:09
@CRobeck CRobeck requested review from ThomasRaoux and removed request for ptillet March 26, 2026 20:09
@ThomasRaoux ThomasRaoux merged commit 82e4bce into main Mar 26, 2026
8 of 9 checks passed
@ThomasRaoux ThomasRaoux deleted the CRobeck-patch-1 branch March 26, 2026 21:40
jvican pushed a commit to jvican/triton that referenced this pull request Mar 27, 2026
…n-lang#9864)

triton-lang#9748 introduced some changes
to the custom ops API and dropped the return value. But the plugin
convention expects operands[0] to be a writable result slot. This adds
that return value for the added op back.
plognjen pushed a commit to plognjen/triton that referenced this pull request Apr 14, 2026
…n-lang#9864)

triton-lang#9748 introduced some changes
to the custom ops API and dropped the return value. But the plugin
convention expects operands[0] to be a writable result slot. This adds
that return value for the added op back.
CRobeck added a commit to CRobeck/triton that referenced this pull request May 5, 2026
…n-lang#9864)

triton-lang#9748 introduced some changes
to the custom ops API and dropped the return value. But the plugin
convention expects operands[0] to be a writable result slot. This adds
that return value for the added op back.
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