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

Fix homogeneous_aggregate? check for aarch64 types #12445

Merged
merged 2 commits into from
Sep 7, 2022

Conversation

mattrberry
Copy link
Contributor

@mattrberry mattrberry commented Sep 4, 2022

Previously we recorded all return and arg types as though they weren't homogeneous aggregates. This minor changes fixes #12407

Side note: This also fixes the ImGui bindings from my emulator :)

image

@Blacksmoke16 Blacksmoke16 added kind:bug A bug in the code. Does not apply to documentation, specs, etc. platform:darwin platform:aarch64 labels Sep 5, 2022
Previously we recorded all return and arg types as though they weren't homogeneous aggregates. This minor changes fixes crystal-lang#12407
@mattrberry
Copy link
Contributor Author

Added a test now that I got the specs to run locally. Apologies for the force push 😬

@mattrberry
Copy link
Contributor Author

Is this not going to make it in 1.5.1?

@straight-shoota
Copy link
Member

@mattrberry That's a good point. I think we can still pull this into 1.5.1. It should've been released today, but I suppose we can push it a bit to incorporate this fix.

@straight-shoota straight-shoota added this to the 1.5.1 milestone Sep 7, 2022
@straight-shoota straight-shoota merged commit 121519a into crystal-lang:master Sep 7, 2022
beta-ziliani pushed a commit to beta-ziliani/crystal that referenced this pull request Sep 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind:bug A bug in the code. Does not apply to documentation, specs, etc. platform:aarch64 platform:darwin
Projects
None yet
Development

Successfully merging this pull request may close these issues.

macOS/arm64 lib binding incorrect
4 participants