-
Notifications
You must be signed in to change notification settings - Fork 100
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
Add Float8E4M3B11FNUZ type support. #1448
Conversation
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.
This LGTM, two test case comments. Also now that I'm seeing that the type changes are in Base.td
, the MHLO integration work may be taken care of already. Any negative test case changes (update expected error message), will be taken care of by the StableHLO oncall.
Will let @ghpvnist review the spec.md
and reference/
changes.
Also regarding the merge conflict, you can overwrite with your changes to |
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 on spec.md
and reference/
changes.
As proposed in [RFC: E4M3B11FNUZ in XLA](https://github.com/openxla/stablehlo/blob/main/rfcs/20230309-e4m3b11.md) (openxla#1308), this change adds support for this type to StableHLO. This includes the type definitions, vhlo, and interpreter support. The testing approach mirrors the Float8E4M3FNUZ tests, since it is also a "non-standard" floating point type supported by StableHLO.
As proposed in RFC: E4M3B11FNUZ in XLA (#1308), this change adds support for these types to StableHLO.
This includes the type definitions, vhlo, and interpreter support. The testing approach mirrors the Float8E4M3FNUZ tests, since it is also a "non-standard" floating point type supported by StableHLO.