Closed yzhang93 closed 3 months ago
Looks good, only minor comments. The PR title is
[Conv2d] Add lowering pipeline and tiling strategy for Conv2d
But it only adds the tiling strategy AFAICT?
Yeah, my intention for this first PR was just to add a new lowering pipeline conv-decompose
. The tiling strategy need to be paired with the pipeline. It doesn't include the building passes for the pipeline yet. Sorry for the confusion.
conv_2d_nchw_fchw
/conv_2d_nhwc_hwcf
). The tiling strategy is following the tile sizes shown in transform dialect, and some dimensions are hardcoded for now. We can explore more on the tiling strategy once simple example working e2e.