Skip to content

[BACKEND] run remove backward prop until a fix point#8776

Merged
ThomasRaoux merged 1 commit into
triton-lang:mainfrom
ThomasRaoux:fixed_point_backward_prop
Nov 20, 2025
Merged

[BACKEND] run remove backward prop until a fix point#8776
ThomasRaoux merged 1 commit into
triton-lang:mainfrom
ThomasRaoux:fixed_point_backward_prop

Conversation

@ThomasRaoux
Copy link
Copy Markdown
Collaborator

No description provided.

@ThomasRaoux ThomasRaoux force-pushed the fixed_point_backward_prop branch from 51893c4 to ce1489b Compare November 19, 2025 23:01
@ThomasRaoux ThomasRaoux force-pushed the fixed_point_backward_prop branch from ce1489b to b7e1eef Compare November 19, 2025 23:04
// CHECK: %[[F:.+]]:3 = scf.for {{.*}} -> (tensor<128xf32, #ttg.slice<{dim = 1, parent = #blocked}>>, tensor<128xf32, #ttg.slice<{dim = 1, parent = #blocked}>>, tensor<128xf32, #ttg.slice<{dim = 1, parent = #mma}>>)
// FIXME: The optimal number of conversions should be 4.
// CHECK-COUNT-5: convert_layout
// CHECK: %[[F:.+]]:3 = scf.for {{.*}} -> (tensor<128xf32, #ttg.slice<{dim = 1, parent = #blocked}>>, tensor<128xf32, #ttg.slice<{dim = 1, parent = #blocked}>>, tensor<128xf32, #ttg.slice<{dim = 1, parent = #blocked}>>)
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Well this is a good sign I guess

Mogball added a commit that referenced this pull request Nov 20, 2025
…+join and remove `CatOp`" (#8777)

Reverts #8769

This depends on #8776 but
since it's a significant change, revert it so we can integrate them
individually.
@ThomasRaoux ThomasRaoux marked this pull request as ready for review November 20, 2025 00:40
@ThomasRaoux ThomasRaoux requested a review from ptillet as a code owner November 20, 2025 00:40
@ThomasRaoux ThomasRaoux changed the title [WIP] Try to run remove backward prop until a fix point [BACKEND] run remove backward prop until a fix point Nov 20, 2025
@ThomasRaoux ThomasRaoux merged commit 85400f8 into triton-lang:main Nov 20, 2025
9 checks passed
tmoreau89 pushed a commit to tmoreau89/triton that referenced this pull request Dec 1, 2025
…+join and remove `CatOp`" (triton-lang#8777)

Reverts triton-lang#8769

This depends on triton-lang#8776 but
since it's a significant change, revert it so we can integrate them
individually.
tmoreau89 pushed a commit to tmoreau89/triton that referenced this pull request Dec 1, 2025
meta-codesync Bot pushed a commit to facebookexperimental/triton that referenced this pull request Apr 6, 2026
…istic by using permute+reshape+join and remove `CatOp`" (#8777)'

Summary:
This is a cherry-pick of an upstream PR: triton-lang/triton#8777

Upstream commit message:
```
> Revert "[Triton] Make `tl.cat` deterministic by using permute+reshape+join and remove `CatOp`" (#8777)

> Reverts triton-lang/triton#8769

> This depends on triton-lang/triton#8776 but
> since it's a significant change, revert it so we can integrate them
> individually.
```

***Do not remove the following line from this commit***
Reactor Cherry-pick Revision: e7fb841
 ---

This diff was generated by running:
```
buck run fbcode//triton/tools/reactor:reactor -- cherrypick --num-commits 1 --no-submit
```

Reviewed By: dshi7

Differential Revision: D99343063

fbshipit-source-id: b7f68732787eaf200639f89adf5ac8b80d69a9b7
meta-codesync Bot pushed a commit to facebookexperimental/triton that referenced this pull request Apr 6, 2026
…l a fix point (#8776)'

Summary:
This is a cherry-pick of an upstream PR: triton-lang/triton#8776

Upstream commit message:
```
> [BACKEND] run remove backward prop until a fix point (#8776)
```

***Do not remove the following line from this commit***
Reactor Cherry-pick Revision: 85400f8
 ---

This diff was generated by running:
```
buck run fbcode//triton/tools/reactor:reactor -- cherrypick --num-commits 1 --no-submit
```

Reviewed By: dshi7

Differential Revision: D99343055

fbshipit-source-id: 778209c25a793286550f1866ea46042bfe715692
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