Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@lialan
Copy link
Contributor

@lialan lialan commented Oct 30, 2025

This patch is required for dropping the revert of: llvm/llvm-project@fcf79e5

Closes #22171

@lialan lialan force-pushed the users/lialan/revert_llvm branch 5 times, most recently from d166421 to c8a9f24 Compare November 4, 2025 13:51
@lialan lialan force-pushed the users/lialan/revert_llvm branch 7 times, most recently from 185ac83 to 5e54a67 Compare November 5, 2025 15:30
@lialan lialan changed the title [Do not merge] Experiment with reverting [Bug] Fix tileDispatchUsingSCFForOp tiling decisions Nov 5, 2025
@lialan lialan marked this pull request as ready for review November 5, 2025 16:33
@lialan lialan force-pushed the users/lialan/revert_llvm branch from 1776b60 to f53376d Compare November 6, 2025 12:50
When deciding skip trivial tiling dimensions, the function should
consider the cases where workgroup size is larger than tile size.
In such cases, we should not simply fold the tiling as it is wrong.

Unfortunately `tileDispatchUsingSCFForOp` does not have the way to
get the workgroup size, so the fix is to plumb the skip decision
into the function.

This patch is required for dropping the revert of:
llvm/llvm-project@fcf79e5
@lialan lialan force-pushed the users/lialan/revert_llvm branch from f53376d to 3907e71 Compare November 6, 2025 16:03
@lialan lialan changed the title [Bug] Fix tileDispatchUsingSCFForOp tiling decisions Update Lit test checks caused by upstream fcf79e5 Nov 6, 2025
Copy link
Contributor

@hanhanW hanhanW left a comment

Choose a reason for hiding this comment

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

Thanks! I just added closes #issue to the PR description.

@hanhanW hanhanW merged commit a88e057 into main Nov 6, 2025
47 of 49 checks passed
@hanhanW hanhanW deleted the users/lialan/revert_llvm branch November 6, 2025 17:37
bangtianliu pushed a commit to bangtianliu/iree that referenced this pull request Nov 19, 2025
pstarkcdpr pushed a commit to pstarkcdpr/iree that referenced this pull request Nov 28, 2025
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.

Remove revert of https://github.com/llvm/llvm-project/pull/160615 on LLVM integrate

4 participants