Skip to content
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

[libc++] Merge 52e9d80d5db20f23979e409f958736d130387f9e into 12.0.1 #50078

Closed
DimitryAndric opened this issue Jun 16, 2021 · 3 comments
Closed
Assignees
Labels
bugzilla Issues migrated from bugzilla libc++ libc++ C++ Standard Library. Not GNU libstdc++. Not libc++abi.

Comments

@DimitryAndric
Copy link
Collaborator

Bugzilla Link 50734
Resolution FIXED
Resolved on Jun 22, 2021 11:56
Version 12.0
OS All
Blocks #48661
CC @ldionne,@mclow,@tstellar
Fixed by commit(s) 52e9d80 0680e2b

Extended Description

Please merge 52e9d80 into 12.0.1.

This enables libc++'s to be used with gcc, which will otherwise complain with "function body not available" (which is, admittedly, a rather obscure error):

In file included from /workspace/src/contrib/llvm-project/libcxx/src/filesystem/operations.cpp:39:
/workspace/src/contrib/llvm-project/libcxx/include/fstream: In function 'bool std::__1::__fs::filesystem::detail::{anonymous}::copy_file_impl(std::__1::__fs::filesystem::detail::{anonymous}::FileDescriptor&, std::__1::__fs::filesystem::detail::{anonymous}::FileDescriptor&, std::__1::error_code&)':
/workspace/src/contrib/llvm-project/libcxx/include/fstream:1329:6: error: inlining failed in call to always_inline 'void std::__1::basic_ifstream<_CharT, _Traits>::__open(int, std::__1::ios_base::openmode) [with _CharT = char; _Traits = std::__1::char_traits]': function body not available
1329 | void basic_ifstream<_CharT, _Traits>::__open(int __fd,
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/workspace/src/contrib/llvm-project/libcxx/src/filesystem/operations.cpp:778:14: note: called from here
778 | in.__open(read_fd.fd, ios::binary);
| ~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~

@DimitryAndric
Copy link
Collaborator Author

assigned to @ldionne

@ldionne
Copy link
Member

ldionne commented Jun 17, 2021

@tstellar
Copy link
Collaborator

Merged: 0680e2b

@llvmbot llvmbot transferred this issue from llvm/llvm-bugzilla-archive Dec 11, 2021
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugzilla Issues migrated from bugzilla libc++ libc++ C++ Standard Library. Not GNU libstdc++. Not libc++abi.
Projects
None yet
Development

No branches or pull requests

3 participants