Skip to content

Commit

Permalink
[libc++][modules] Fixes exporting named declarations.
Browse files Browse the repository at this point in the history
@chuanqixu noticed std::atomic was not properly exported in the std module.
Investigation showed other named declarations were not exported either. This
fixes the issue.

Depends on D156550

Reviewed By: #libc, philnik

Differential Revision: https://reviews.llvm.org/D156592

(cherry picked from commit e57f6f7)
  • Loading branch information
mordante authored and tru committed Aug 8, 2023
1 parent 992f540 commit edc0ee3
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 6 deletions.
3 changes: 0 additions & 3 deletions libcxx/modules/std/atomic.cppm
Original file line number Diff line number Diff line change
Expand Up @@ -23,9 +23,6 @@ export namespace std {
using std::memory_order_seq_cst;

using std::kill_dependency;
} // namespace std

namespace std {

// [atomics.ref.generic], class template atomic_ref
// [atomics.ref.pointer], partial specialization for pointers
Expand Down
2 changes: 1 addition & 1 deletion libcxx/modules/std/execution.cppm
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ export namespace std {
using std::is_execution_policy_v;
} // namespace std

namespace std::execution {
export namespace std::execution {
// [execpol.seq], sequenced execution policy
using std::execution::sequenced_policy;

Expand Down
4 changes: 2 additions & 2 deletions libcxx/modules/std/filesystem.cppm
Original file line number Diff line number Diff line change
Expand Up @@ -110,11 +110,11 @@ export namespace std::filesystem {
} // namespace std::filesystem

// [fs.path.hash], hash support
namespace std {
export namespace std {
using std::hash;
}

namespace std::ranges {
export namespace std::ranges {
using std::ranges::enable_borrowed_range;
using std::ranges::enable_view;
} // namespace std::ranges

0 comments on commit edc0ee3

Please sign in to comment.