-
Notifications
You must be signed in to change notification settings - Fork 556
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
[StableHLO] Use stablehlo submodule #14094
Conversation
543eea5
to
ee2a2c8
Compare
c6ca08b
to
cd90cbf
Compare
Use the stablehlo submodule to provide stablehlo, since we do not need the 'mhlo' part or mlir-hlo anymore.
cd90cbf
to
c1f7c7a
Compare
The failing lint check is for some existing issues in |
Ping |
CODEOWNERS added lots of reviewers for this. I pruned the list a bit, but it might also help to do that sooner / explicitly say who you expect reviews from. |
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; just need to address Scott's comments there.
Wow this is awesome! Very happy to know that this works seamlessly after openxla/stablehlo#1612. |
Co-authored-by: Scott Todd <scotttodd@google.com>
Awesome, congrats!! |
Use the stablehlo submodule to provide stablehlo, since we do not need the 'mhlo' part or mlir-hlo anymore. Fixes: iree-org#12678 --------- Co-authored-by: Scott Todd <scotttodd@google.com>
Use the stablehlo submodule to provide stablehlo, since we do not need the 'mhlo' part or mlir-hlo anymore.
Fixes: #12678