bytedance / bytedance/byteir

[QUESTION] Convolution Dynamic Shape Support

Open
#498 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
MLIR
Stars
475
Forks
58
PR merge metrics
No merged PRs in 30d

Description

After reading the code in `compiler/lib/Dialect/mhlo/DynamicShapeRegister/Convolution.cpp`, I find there is no `ReifyReturnTyeShapes` function for convolution op. Does convolution op not support symbol relation computing between input and output, or we do it in other pass?

Contributor guide

Open the contributing guide

Research direction

Start by reading compiler/lib/Dialect/mhlo/DynamicShapeRegister/Convolution.cpp and locating the convolution handling around ReifyReturnTyeShapes. Trace whether symbol relations between convolution inputs and outputs are computed in another pass. Done means documenting where this support exists or confirming the missing implementation and its required scope.

Written by the indexing model from the issue text.

Assessment

Domain
compilers
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
20/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.