Skip to content

Conversation

@Lunderberg
Copy link
Contributor

Previously, the legalization was only handled by propagating the dtype of the indices to the transformed indices. As a result, output indices whose value did not depend on the input index would be left with the incorrect dtype.

Previously, the legalization was only handled by propagating the dtype
of the indices to the transformed indices.  As a result, output
indices whose value did not depend on the input index would be left
with the incorrect dtype.
@Lunderberg Lunderberg requested a review from masahi March 23, 2023 20:31
@tvm-bot
Copy link
Collaborator

tvm-bot commented Mar 23, 2023

Thanks for contributing to TVM! Please refer to the contributing guidelines https://tvm.apache.org/docs/contribute/ for useful information and tips. Please request code reviews from Reviewers by @-ing them in a comment.

Generated by tvm-bot

1 similar comment
@tvm-bot
Copy link
Collaborator

tvm-bot commented Mar 23, 2023

Thanks for contributing to TVM! Please refer to the contributing guidelines https://tvm.apache.org/docs/contribute/ for useful information and tips. Please request code reviews from Reviewers by @-ing them in a comment.

Generated by tvm-bot

@masahi masahi merged commit b09e72b into apache:main Mar 24, 2023
@Lunderberg Lunderberg deleted the index_map_legalize_dtype_of_constant branch March 24, 2023 13:07
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.

3 participants