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

[CIR][ABI][AArch64] support for return struct types greater than 128 bits #1027

Merged
merged 4 commits into from
Oct 31, 2024

Conversation

gitoleg
Copy link
Collaborator

@gitoleg gitoleg commented Oct 30, 2024

This PR adds a support for the return values of struct types > 128 bits in size.
As usually, lot's of copy-pasted lines from the original codegen, except the AllocaOp replacement for the return value.

Copy link

github-actions bot commented Oct 30, 2024

✅ With the latest revision this PR passed the C/C++ code formatter.

Copy link
Member

@bcardosolopes bcardosolopes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM with minor change. @sitio-couto anything to add here?

@bcardosolopes
Copy link
Member

Sounds like there are some clang-format issues too

Copy link
Collaborator

@seven-mile seven-mile left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some extra nits on format ; )

@bcardosolopes bcardosolopes merged commit c8f189b into llvm:main Oct 31, 2024
6 checks passed
lanza pushed a commit that referenced this pull request Nov 5, 2024
…bits (#1027)

This PR adds a support for the return values of struct types > 128 bits
in size.
As usually, lot's of copy-pasted lines from the original codegen, except
the `AllocaOp` replacement for the return value.
lanza pushed a commit that referenced this pull request Mar 18, 2025
…bits (#1027)

This PR adds a support for the return values of struct types > 128 bits
in size.
As usually, lot's of copy-pasted lines from the original codegen, except
the `AllocaOp` replacement for the return value.
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.

None yet

3 participants