Repository navigation
Refactor the Cute-DSL AR fusion to support DeepseekV2 archs (GLM-5.3, etc.) #39816
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
Merged
Merged
Changes from 30 commits
Commits
Show all changes
35 commits
Select commit
Hold shift + click to select a range
b1ccde1
[Comm] Share the CuTe DSL AR fusion core and wire it for DeepSeek-V3 …
b8zhong 16eb74d
[Comm] Trim the CuTe DSL fusion comments to the facts
b8zhong 14296e3
[Comm] Drop the CuTe DSL workspace instance-count env var
b8zhong 7a90133
[Comm] Delete dead surface in the CuTe DSL AR fusion
b8zhong 6401718
[Comm] Bound the deferred MoE finalize by token count and absorb the …
b8zhong 41af7ef
[Comm] Add the CuTe DSL AR fusion measurement record
b8zhong 412b268
[Comm] Fix two wording defects in the measurement record
b8zhong e4e5d5e
[Comm] Report decode throughput instead of inter-token latency
b8zhong b3c396a
[Comm] Show the arms as a percentage gain over stock
b8zhong c25813f
Merge remote-tracking branch 'origin/main' into brayden/cutedsl-ar-fu…
148e75f
Drop two DSA/DCP tests that do not belong to this branch
4457499
Delete the unreachable should_use_finalize base stub
cca1513
Record one deferral flag instead of two
1a1536e
Delete Qwen3.5's tautological communicator check
29399f3
Separate incoming from outgoing post-MoE reduction eligibility
de189bb
Keep the deferred handoff inside the dual-stream op's tensor contract
7a342f9
Warn on the removed CuTe DSL fusion switch and fix the cookbook recipes
f1116b0
Drop the measurement record from the tree
bd63a99
Spell the fusion backend value cutedsl
8fa2317
Follow the repository rules in the CuTe DSL fusion
448f9bf
Move the dual-stream op contract test to the CPU fusion suite
4f1ad89
Merge remote-tracking branch 'origin/main' into brayden/cutedsl-ar-fu…
b8zhong 92f9b83
Sync changes
bf920a7
Drop the cookbook changes
0e7b7b0
Restore the deferred MoE finalize token bound
2b365c8
Sync
0360803
Merge branch 'main' into brayden/cutedsl-ar-fusion-shared-core
mmangkad 781d629
Merge remote-tracking branch 'origin/main' into brayden/cutedsl-ar-fu…
607a0c1
[MoE] Allow GlmMoeDsaForCausalLM for the FlashInfer MegaMoE backend
b8zhong 21ba00e
Merge remote-tracking branch 'origin/main' into brayden/cutedsl-ar-fu…
61ab151
Merge remote-tracking branch 'origin/main' into brayden/cutedsl-ar-fu…
d0c1bc3
Address reviewers' feedback on the CuTe DSL AR fusion
e827f19
Address the second review round on the CuTe DSL AR fusion
1562d64
Run the GLM-5.2 NVFP4 TP MTP test on the CuTe DSL fusion
7edb3c7
Merge branch 'main' into brayden/cutedsl-ar-fusion-shared-core
mmangkad File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we are only turning on moe deferred finalize for decode only right? Btw this number feels a little hardcoded / unintuitive.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes. The 192 was measured by @b8zhong on B300, and I confirmed on GB300 that it's reasonable (the crossover is around 192–223), so I think we can keep it