-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
[Target][Legalization]Add Tir Level Legalization Function Registration And Update Intrinsic Lowering Pass #7936
Conversation
IMHO, the intrinsic lowering pass logic is already in |
@zxybazh The PR looks good. Please retrigger the CI :-) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. I don't have much to comment unlike the previous two :-)
Thanks for the contribution!
This PR looks good to me. @tqchen @anijain2305 @zhiics Please let me know if there are any further comments. |
…n And Update Intrinsic Lowering Pass (apache#7936)
…n And Update Intrinsic Lowering Pass (apache#7936)
…n And Update Intrinsic Lowering Pass (apache#7936)
…n And Update Intrinsic Lowering Pass (apache#7936)
…n And Update Intrinsic Lowering Pass (apache#7936)
This PR introduces tir level op legalization function registration to current tvm stack and injects the legalization function when no intrinsic lowering function is available. Currently the legalization functions are target-dependent. Previous legalization functions are migrated to the legelization function registry as well.