Skip to content

Conversation

@bangtianliu
Copy link
Contributor

@bangtianliu bangtianliu commented Nov 28, 2025

Context

padding attribute: A general padding configuration used for most operations (matmul, IGEMM, and other linalg operations). Specified in IGEMM/GEMM dimension space (after convolution is transformed to IGEMM). Used by GPUPadOperandsPass to pad operations in their transformed GEMM form.

padding_conv attribute: A specialized padding configuration for convolutions before IGEMM transformation. Specified in the original convolution dimension space (before transformation). Used by GPUPadConvsPass to pad convolution operations in their original form by mapping IGEMM padding sizes back to convolution dimensions.

Key difference: padding works in GEMM space (M, N, K dimensions), while padding_conv works in the original convolution space (batch, spatial, channel dimensions).

Reference PR from IREE side: iree-org/iree#21470

What this PR does

This PR adds support for the padding_conv attribute for convolution operations when using the IGEMM transformation path. This enables padding convolutions in their original dimension space before IGEMM transformation.

Issue: #2687

@bangtianliu bangtianliu marked this pull request as draft November 28, 2025 18:39
@bangtianliu bangtianliu force-pushed the padding_conv branch 2 times, most recently from 4c2cb38 to 6e904d5 Compare November 28, 2025 21:05
@bangtianliu bangtianliu force-pushed the padding_conv branch 2 times, most recently from ac90ffe to 9a449fc Compare December 1, 2025 17:54
@bangtianliu bangtianliu force-pushed the padding_conv branch 7 times, most recently from 4ed0682 to 1611fc2 Compare December 2, 2025 00:43
@bangtianliu bangtianliu marked this pull request as ready for review December 2, 2025 00:59
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.

1 participant