Skip to content

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

Merged
merged 1 commit into from
Nov 25, 2024
Merged

Add extendhfxf2 into compiler rt #113897

merged 1 commit into from
Nov 25, 2024

Conversation

biabbas
Copy link
Contributor

@biabbas biabbas commented Oct 28, 2024

Fixes #105181
Retry of pr #109090 and #111099.
ping @arichardson

Regards.

@biabbas biabbas marked this pull request as draft October 28, 2024 11:58
@biabbas biabbas force-pushed the extendhfxf2 branch 2 times, most recently from 4393971 to 270ec84 Compare October 28, 2024 12:12
@biabbas biabbas marked this pull request as ready for review October 28, 2024 12:13
@biabbas biabbas force-pushed the extendhfxf2 branch 5 times, most recently from 8ddd5b9 to b80ef73 Compare October 30, 2024 09:20
Copy link
Member

@arichardson arichardson left a 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.

@biabbas biabbas force-pushed the extendhfxf2 branch 3 times, most recently from 20116c2 to ba8237d Compare November 6, 2024 09:46
Copy link
Member

@arichardson arichardson left a 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.

Copy link

github-actions bot commented Nov 7, 2024

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

Copy link
Member

@arichardson arichardson left a 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]>
@biabbas
Copy link
Contributor Author

biabbas commented Nov 25, 2024

I've updated the branch for merge.

@arichardson arichardson merged commit a5dd646 into llvm:main Nov 25, 2024
7 checks passed
@rorth
Copy link
Collaborator

rorth commented Nov 25, 2024

This broke the Solaris/amd64 bot again.

@biabbas
Copy link
Contributor Author

biabbas commented Nov 26, 2024

This broke the Solaris/amd64 bot again.

I'll update the test case asap.

#include <stdio.h>

#include "int_lib.h"

Copy link
Member

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};
Copy link
Member

Choose a reason for hiding this comment

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

Can drop this

@biabbas biabbas mentioned this pull request Nov 26, 2024
@mstorsjo
Copy link
Member

arichardson added a commit that referenced this pull request Nov 26, 2024
Fix changes in #113897

Co-authored-by: Alex Richardson <[email protected]>
@biabbas biabbas deleted the extendhfxf2 branch November 27, 2024 04:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[builtins] Missing __extendhfxf2 and __truncxfhf2
5 participants