Skip to content
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

feat: add f32 and f64 types #40

Merged
merged 7 commits into from
Jan 9, 2025

Conversation

dshaaban01
Copy link
Contributor

Added f32 and f64 types + f32 and f64 attribute types.

Added tests for f32 and f64 types but NOT for attributes. Waiting on this PR to be merged and then will add those tests to project.mlir test file (similarly to integer types)! @ingomueller-net

@dshaaban01 dshaaban01 changed the title feat: add f32 and f64 types and tests feat: add f32 and f64 types Dec 27, 2024
@dshaaban01 dshaaban01 force-pushed the dalia/types/floats branch 3 times, most recently from fae0b91 to 810f6bd Compare January 6, 2025 15:26
@dshaaban01
Copy link
Contributor Author

dshaaban01 commented Jan 6, 2025

Note: 2 missing test files, have weird failures I will analyze later today.

Edit: Done

@ingomueller-net
Copy link
Collaborator

(I'll let you rebase the remaining PRs before taking a look. They'll have conflicts like this one anyway; hopefully, I can merge most of the right away after the rebase.)

@dshaaban01
Copy link
Contributor Author

Rebase floats done.

@ingomueller-net ingomueller-net merged commit 9731f91 into substrait-io:main Jan 9, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants