-
Notifications
You must be signed in to change notification settings - Fork 10.8k
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 support for DFP IR type. #69718
Add support for DFP IR type. #69718
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 looks right. I've got a bunch of minor comments, but only a couple of things of any significance.
Does anyone know what is the Test Documentation build fail? I haven't seen this before. |
Hi, Zahira. We haven't been pushing DFP changes to upstream yet. Could you submit a PR to the dfp branch of my fork instead? |
hmm! I thought that's what I had done! |
a15d72b
to
583eea1
Compare
Was trying to pull this branch into my fork to debug a CI failure for the documentation build and annoyingly |
Add decimal32, decimal64 and decimal128 IR types.
Still to be done:
APFloat
Constant Folding