Skip to content

Conversation

@yzh119
Copy link
Member

@yzh119 yzh119 commented Jun 29, 2023

Motivation

Currently, the legalization passes would be triggered whether the hardware supports it or not.
This PR disables the pass when the pass context sets target_support_bf16 to True.

cc @tqchen @Hzfengsy @vinx13

@tvm-bot
Copy link
Collaborator

tvm-bot commented Jun 29, 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

@yzh119 yzh119 marked this pull request as ready for review June 29, 2023 14:35
@yzh119 yzh119 changed the title [WIP][Runtime] Disable legalization passes when hardware supports bf16 [Runtime] Disable legalization passes when hardware supports bf16 Jun 29, 2023
@github-actions github-actions bot requested review from Hzfengsy and tqchen June 29, 2023 14:38
@github-actions github-actions bot requested a review from vinx13 July 3, 2023 08:58
junrushao pushed a commit that referenced this pull request Jul 21, 2023
The PR #15183 adds ml_dtypes as a dependency of TVM, however, some of the CI docker images do not have ml_dtypes installed, the PR fixes the issue.
@tqchen tqchen closed this May 8, 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.

3 participants