-
Notifications
You must be signed in to change notification settings - Fork 116
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
Create a plan for CHLO #602
Comments
When writing the previous comment, I was convinced that we should have a CHLO spec as well, similarly to the StableHLO spec. However, after thinking about this further, I'm realizing that this is not necessarily the case. One seemingly viable option is treating CHLO similarly to how client HLO is treated - i.e. helper functionality that is useful for producers but is not reified in the operation set. This way, the StableHLO project will have less stuff to spec and less stuff to maintain, and StableHLO consumers will have less stuff to support. This may run into logistical difficulties, because chlo-legalize-to-hlo currently lives in the MLIR-HLO repository, and it will likely be desirable to move it to this repository instead if we choose this course of action, but at the time of writing I think it's a viable alternative to consider. |
Open questions:
The text was updated successfully, but these errors were encountered: