Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[mlir][TilingInterface] Make the tiling set tile sizes function use OpFoldResult. #66566

Merged
merged 1 commit into from
Sep 19, 2023

Conversation

MaheshRavishankar
Copy link
Contributor

No description provided.

SmallVector<OpFoldResult> &sizes) {
static SmallVector<scf::ForOp> generateTileLoopNest(
OpBuilder &builder, Location loc, ArrayRef<Range> loopRanges,
ArrayRef<OpFoldResult> tileSizeVals, SmallVector<OpFoldResult> &offsets,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: tileSizeVals -> tileSizes.

Copy link
Contributor

@chelini chelini left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, looks good to me.

@MaheshRavishankar MaheshRavishankar force-pushed the mahesh_use_scfforall branch 3 times, most recently from 2cbb94a to 24738d8 Compare September 18, 2023 17:38
Copy link
Contributor

@nicolasvasilache nicolasvasilache left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LG

MaheshRavishankar added a commit to iree-org/iree that referenced this pull request Sep 18, 2023
MaheshRavishankar added a commit to iree-org/iree that referenced this pull request Sep 18, 2023
MaheshRavishankar added a commit to MaheshRavishankar/iree that referenced this pull request Sep 18, 2023
MaheshRavishankar added a commit to MaheshRavishankar/iree that referenced this pull request Sep 18, 2023
MaheshRavishankar added a commit to iree-org/iree that referenced this pull request Sep 18, 2023
MaheshRavishankar added a commit to iree-org/iree that referenced this pull request Sep 18, 2023
MaheshRavishankar added a commit to iree-org/iree that referenced this pull request Sep 18, 2023
MaheshRavishankar added a commit to iree-org/iree that referenced this pull request Sep 18, 2023
@MaheshRavishankar MaheshRavishankar merged commit 170a25a into llvm:main Sep 19, 2023
@MaheshRavishankar MaheshRavishankar deleted the mahesh_use_scfforall branch September 19, 2023 00:19
ZijunZhaoCCK pushed a commit to ZijunZhaoCCK/llvm-project that referenced this pull request Sep 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants