-
-
Notifications
You must be signed in to change notification settings - Fork 2.9k
#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
Conversation
Congrats on your first PR btw 🙂 |
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.
Error message typos.
Sorry for being unclear, my comment on |
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.
Looks okay to me. @jacobly0 were all of your concerns addressed?
This is my first pull request, feedback welcome.
Closes #23909.