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

Reapply "[Flow] Convert from tensor.cast to flow.tensor.reshape" #18351

Merged
merged 4 commits into from
Aug 26, 2024

Conversation

nirvedhmeshram
Copy link
Contributor

@nirvedhmeshram nirvedhmeshram commented Aug 26, 2024

The first commit relands the canonicalization that was reverted due to an ARM issue llvm/llvm-project#105869 (and more recently RISCV issue).

The second commit disables the failing tests for the respective backend.

@nirvedhmeshram nirvedhmeshram force-pushed the users/nirvedhmeshram/reland_tensorcast branch from 7f85e69 to bf047c5 Compare August 26, 2024 16:14
@nirvedhmeshram nirvedhmeshram changed the title Reapply "[Flow] Convert from tensor.cast to flow.tensor.reshape ( #18256)" (#18331) Reapply "[Flow] Convert from tensor.cast to flow.tensor.reshape" Aug 26, 2024
@nirvedhmeshram nirvedhmeshram force-pushed the users/nirvedhmeshram/reland_tensorcast branch 5 times, most recently from c7fa345 to 9e9f10d Compare August 26, 2024 16:50
@nirvedhmeshram nirvedhmeshram force-pushed the users/nirvedhmeshram/reland_tensorcast branch from 9e9f10d to c51b165 Compare August 26, 2024 16:58
@nirvedhmeshram nirvedhmeshram force-pushed the users/nirvedhmeshram/reland_tensorcast branch from c51b165 to dcc5e11 Compare August 26, 2024 17:01
@nirvedhmeshram
Copy link
Contributor Author

Looks like there is another backend failure is RISCV for a conv test, after discussing with @MaheshRavishankar , we decided to disable this test as well.

@nirvedhmeshram nirvedhmeshram force-pushed the users/nirvedhmeshram/reland_tensorcast branch from bd41466 to 8c8e02c Compare August 26, 2024 17:59
@nirvedhmeshram
Copy link
Contributor Author

The test_einsum_inner_prod failure in the ONXX suite is unrelated so landing this PR

@nirvedhmeshram nirvedhmeshram merged commit d2e74cd into main Aug 26, 2024
40 of 42 checks passed
@nirvedhmeshram nirvedhmeshram deleted the users/nirvedhmeshram/reland_tensorcast branch August 26, 2024 19:44
IanWood1 added a commit to IanWood1/iree that referenced this pull request Oct 14, 2024
IanWood1 added a commit that referenced this pull request Oct 16, 2024
Revert `tensor.cast` to `flow.tensor.reshape` conversion because this is occurring too early. This happens as early as `GlobalOptimization` since Flow CanonicalizerPass is used there. Also, stop cloning `tensor::CastOp`s into dispatches.

Reverted pr: #18351 Original Issue: #18229
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.

2 participants