-
Notifications
You must be signed in to change notification settings - Fork 14.3k
Add extendhfxf2 into compiler rt #113897
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
Add extendhfxf2 into compiler rt #113897
Conversation
4393971
to
270ec84
Compare
8ddd5b9
to
b80ef73
Compare
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.
I'd recommend adding DST_80 support fp_extend.h to make this slightly cleaner.
20116c2
to
ba8237d
Compare
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.
Minor not, but LGTM with those changes.
✅ With the latest revision this PR passed the C/C++ code formatter. |
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.
This looks good to me, just one question about i386 - maybe safer to add the #if
Co-authored-by: Alexander Richardson <[email protected]>
I've updated the branch for merge. |
This broke the Solaris/amd64 bot again. |
I'll update the test case asap. |
#include <stdio.h> | ||
|
||
#include "int_lib.h" | ||
|
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.
#include "fp_test.h"
return ret; | ||
} | ||
|
||
char assumption_1[sizeof(_Float16) * CHAR_BIT == 16] = {0}; |
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.
Can drop this
This also broke the compiler-rt tests on mingw, again: https://github.com/mstorsjo/llvm-mingw/actions/runs/12021713343/job/33522994914 and https://github.com/mstorsjo/llvm-mingw/actions/runs/12021713343/job/33522995313 I can test the incremental fix in #117665. |
Fix changes in #113897 Co-authored-by: Alex Richardson <[email protected]>
Fixes #105181
Retry of pr #109090 and #111099.
ping @arichardson
Regards.