[AIMIGRAPHX-1005] [AIMIGRAPHX-1227] [AIMIGRAPHX-1228] - #5114
Conversation
…hack in flash decoding
…r broadcast shapes, instead uses BFS to do a topological rebuild, waiting until deps are ready
There was a problem hiding this comment.
Pull request overview
This PR strengthens the fuse_attention flash-decoding path by making flash-decoding submodule rebuilding more robust in the presence of additional submodule inputs (e.g., masks), constants/materialized literals, and non-trivial instruction ordering. It also expands the unit tests to cover these additional graph shapes and edge cases, and makes existing tests less sensitive to host environment configuration.
Changes:
- Add test coverage for flash decoding with embedded
@literal, extra@parammask inputs, unary ops affecting broadcast shape inference, outline nodes, and rebuild ordering constraints. - Update flash-decoding submodule rebuild to handle
@literal/@outline, extra score-shaped params, and dependency-aware rebuild ordering (incl. broadcast shape inference). - Ensure certain tests run
fuse_attentionwithout inheriting flash-decoding settings from environment variables.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
test/fuse_attention.cpp |
Adds new flash-decoding tests and introduces an “isolated” pass runner to avoid environment-dependent behavior. |
src/fuse_attention.cpp |
Improves flash-decoding submodule reconstruction (params handling, literals/outlines, topo-ish rebuild, broadcast shape handling). |
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #5114 +/- ##
===========================================
- Coverage 93.26% 93.22% -0.04%
===========================================
Files 623 623
Lines 33097 33191 +94
===========================================
+ Hits 30866 30942 +76
- Misses 2231 2249 +18
🚀 New features to boost your workflow:
|
Regressions detected 🔴 * No develop baseline was found for this PR's branch point; compared against the latest available develop run instead. |
|
Motivation
Technical Details
Changelog Category
Add a
CHANGELOG.mdentry for any option other thanNot ApplicableFollow the LLVM AI Tool Use Policy for contributions using AI.