Skip to content

[builtins] Support QUAD with native long double #77981

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

Closed
wants to merge 3 commits into from

Conversation

perry-ca
Copy link
Contributor

Redo #77554 to solve build issue found in #77880.

Copy link

github-actions bot commented Jan 12, 2024

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

@perry-ca perry-ca changed the title support QUAD with native long double [builtins] Support QUAD with native long double Jan 12, 2024
@perry-ca
Copy link
Contributor Author

@arichardson any idea why the build failed? It appears to be in some tests for dlopen().

@arichardson
Copy link
Member

I'll try testing this for various architectures tomorrow and if that works I'll go ahead and merge it.

@arichardson
Copy link
Member

I am getting some errors when cross-building for PPC64, will do some digging to see how the ifdefs need to be adjusted.

@perry-ca
Copy link
Contributor Author

perry-ca commented Jan 16, 2024 via email

@perry-ca
Copy link
Contributor Author

@arichardson has addressed the build issues in #82208. This PR is superseded by this PR.

@perry-ca perry-ca closed this Feb 21, 2024
@perry-ca perry-ca deleted the perry/redo-quad-non-tf-mode branch April 24, 2024 20:50
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.

3 participants