Skip to content

[Fix][Relax] Preserve tensor-derived symbols during fusion - #20139

Merged
tlopex merged 3 commits into
apache:mainfrom
akaashrp:upstream/relax-fusion-symbolic-args
Aug 28, 2026
Merged

tlopex merged 3 commits into
apache:mainfrom
akaashrp:upstream/relax-fusion-symbolic-args

Conversation

@akaashrp

Copy link
Copy Markdown
Contributor

FuseOps may lift a symbolic primitive argument into an independent scalar parameter before the complete fused-function boundary is known. This can sever its relationship with tensor or shape types, making the result argument-order dependent and potentially producing an invalid or redundant ABI. This PR classifies symbolic primitive arguments after collecting the full fused-function boundary, preserving their relationship with tensor and shape types regardless of argument order.

@yongwww yongwww closed this Aug 17, 2026
@yongwww yongwww reopened this Aug 17, 2026
@akaashrp

Copy link
Copy Markdown
Contributor Author

@tvm-bot rerun

@akaashrp
akaashrp requested a review from tlopex August 25, 2026 02:42
Comment thread src/relax/transform/fuse_ops.cc Outdated
Comment thread src/relax/transform/fuse_ops.cc Outdated
@akaashrp
akaashrp force-pushed the upstream/relax-fusion-symbolic-args branch from e607c58 to a9137fc Compare August 27, 2026 18:46
@tlopex
tlopex merged commit 06a8102 into apache:main Aug 28, 2026
8 checks passed
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