Skip to content

#23909 adds __addvsi3, __subvsi3, __mulvsi3, subvdi3 to compiler_rt #24000

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

Merged
merged 13 commits into from
Jun 1, 2025

Conversation

hi7
Copy link
Contributor

@hi7 hi7 commented May 27, 2025

This is my first pull request, feedback welcome.

Closes #23909.

@alexrp alexrp self-assigned this May 27, 2025
@andrewrk
Copy link
Member

Congrats on your first PR btw 🙂

Copy link
Contributor

@jayschwa jayschwa left a comment

Choose a reason for hiding this comment

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

Error message typos.

@jacobly0
Copy link
Member

Sorry for being unclear, my comment on __addvsi3 applies to all the functions in this PR. (Also the subs are sorted backwards)

Copy link
Member

@alexrp alexrp left a comment

Choose a reason for hiding this comment

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

Looks okay to me. @jacobly0 were all of your concerns addressed?

@alexrp alexrp merged commit 0386730 into ziglang:master Jun 1, 2025
9 checks passed
@alexrp alexrp removed their assignment Jun 1, 2025
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.

compiler_rt missing symbols: __addvsi3, __subvsi3, __mulvsi3, __subvdi3
5 participants