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

[GlobalOpt] Disable pack->expand_shape propagation. #17739

Merged
merged 2 commits into from
Jun 25, 2024

Conversation

hanhanW
Copy link
Contributor

@hanhanW hanhanW commented Jun 25, 2024

It is not enabled until the upstream commit: llvm/llvm-project@a945f55

The upstream commit triggers a bug in IREE, so the revision disables the propagation.

See #17734 for more details.

It is not enabled until the upstream commit: llvm/llvm-project@a945f55

The upstream commit triggers a bug in IREE, so the revision disables the
propagation.

Signed-off-by: hanhanW <[email protected]>
Copy link
Member

@ScottTodd ScottTodd left a comment

Choose a reason for hiding this comment

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

The crash I was seeing went away with this.

Looks like some tests need updating though.

Signed-off-by: hanhanW <[email protected]>
@hanhanW
Copy link
Contributor Author

hanhanW commented Jun 25, 2024

The crash I was seeing went away with this.

Looks like some tests need updating though.

I need to check the benchmark results. We also need a better way to control the propagation, I'll help update it.

@hanhanW hanhanW added benchmarks:x86_64 Run default x86_64 benchmarks benchmarks:android-cpu Run default Android CPU benchmarks labels Jun 25, 2024
@hanhanW hanhanW marked this pull request as ready for review June 25, 2024 18:51
@hanhanW
Copy link
Contributor Author

hanhanW commented Jun 25, 2024

Merging the PR to unblock other work. I'll follow up on this.

@hanhanW hanhanW merged commit 5fcdddb into iree-org:main Jun 25, 2024
66 of 68 checks passed
@hanhanW hanhanW deleted the issue17734 branch June 25, 2024 20:15
LLITCHEV pushed a commit to LLITCHEV/iree that referenced this pull request Jul 30, 2024
It is not enabled until the upstream commit:
llvm/llvm-project@a945f55

The upstream commit triggers a bug in IREE, so the revision disables the
propagation.

See iree-org#17734 for more details.

---------

Signed-off-by: hanhanW <[email protected]>
Signed-off-by: Lubo Litchev <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
benchmarks:android-cpu Run default Android CPU benchmarks benchmarks:x86_64 Run default x86_64 benchmarks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants