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++][ranges] export std::ranges::range_adaptor_closure #89793

Merged
merged 1 commit into from
Apr 23, 2024

Conversation

xiaoyang-sde
Copy link
Member

Abstract

This pull request exports the std::ranges::range_adaptor_closure class template implemented in #89148.

// [range.adaptor.object], range adaptor objects
template<class D>
  requires is_class_v<D> && same_as<D, remove_cv_t<D>>
class range_adaptor_closure { };

Reference

@xiaoyang-sde xiaoyang-sde requested a review from a team as a code owner April 23, 2024 16:49
@llvmbot llvmbot added the libc++ libc++ C++ Standard Library. Not GNU libstdc++. Not libc++abi. label Apr 23, 2024
@llvmbot
Copy link
Member

llvmbot commented Apr 23, 2024

@llvm/pr-subscribers-libcxx

Author: Xiaoyang Liu (xiaoyang-sde)

Changes

Abstract

This pull request exports the std::ranges::range_adaptor_closure class template implemented in #89148.

// [range.adaptor.object], range adaptor objects
template&lt;class D&gt;
  requires is_class_v&lt;D&gt; &amp;&amp; same_as&lt;D, remove_cv_t&lt;D&gt;&gt;
class range_adaptor_closure { };

Reference


Full diff: https://github.com/llvm/llvm-project/pull/89793.diff

1 Files Affected:

  • (modified) libcxx/modules/std/ranges.inc (+3-1)
diff --git a/libcxx/modules/std/ranges.inc b/libcxx/modules/std/ranges.inc
index 82c7d99f8979a8..7d215867a4317f 100644
--- a/libcxx/modules/std/ranges.inc
+++ b/libcxx/modules/std/ranges.inc
@@ -138,8 +138,10 @@ export namespace std {
     }
 #endif // _LIBCPP_HAS_NO_LOCALIZATION
 
+#if _LIBCPP_STD_VER >= 23
     // [range.adaptor.object], range adaptor objects
-    // using std::ranges::range_adaptor_closure;
+    using std::ranges::range_adaptor_closure;
+#endif
 
     // [range.all], all view
     namespace views {

@ldionne ldionne merged commit 3a9d8cd into llvm:main Apr 23, 2024
52 of 53 checks passed
@xiaoyang-sde xiaoyang-sde deleted the range_adaptors branch April 23, 2024 17:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
libc++ libc++ C++ Standard Library. Not GNU libstdc++. Not libc++abi.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants