[libcxx-commits] [PATCH] D152617: [libc++][spaceship] P1614R2: Added `operator==` to `slice`
Hristo Hristov via Phabricator via libcxx-commits
libcxx-commits at lists.llvm.org
Sat Jun 10 06:32:14 PDT 2023
H-G-Hristov updated this revision to Diff 530202.
H-G-Hristov added a comment.
Try to fix CI
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D152617/new/
https://reviews.llvm.org/D152617
Files:
libcxx/include/valarray
libcxx/test/std/numerics/numarray/class.slice/slice.ops/slice.ops.pass.cpp
Index: libcxx/test/std/numerics/numarray/class.slice/slice.ops/slice.ops.pass.cpp
===================================================================
--- /dev/null
+++ libcxx/test/std/numerics/numarray/class.slice/slice.ops/slice.ops.pass.cpp
@@ -0,0 +1,51 @@
+//===----------------------------------------------------------------------===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
+//
+//===----------------------------------------------------------------------===//
+
+// <valarray>
+
+// class slice;
+
+// friend bool operator==(const slice& x, const slice& y);
+
+#include <valarray>
+#include <cassert>
+
+int main(int, char**) {
+ {
+ std::slice s1;
+ std::slice s2;
+
+ assert(s1 == s2);
+ }
+ {
+ std::slice s1{1, 2, 3};
+ std::slice s2{1, 2, 3};
+
+ assert(s1 == s2);
+ }
+ {
+ std::slice s1{0, 2, 3};
+ std::slice s2{1, 2, 3};
+
+ assert(s1 != s2);
+ }
+ {
+ std::slice s1{1, 0, 3};
+ std::slice s2{1, 2, 3};
+
+ assert(s1 != s2);
+ }
+ {
+ std::slice s1{1, 2, 0};
+ std::slice s2{1, 2, 3};
+
+ assert(s1 != s2);
+ }
+
+ return 0;
+}
Index: libcxx/include/valarray
===================================================================
--- libcxx/include/valarray
+++ libcxx/include/valarray
@@ -116,6 +116,8 @@
size_t start() const;
size_t size() const;
size_t stride() const;
+
+ friend bool operator==(const slice& x, const slice& y); // since C++20
};
template <class T>
@@ -402,6 +404,14 @@
_LIBCPP_INLINE_VISIBILITY size_t start() const {return __start_;}
_LIBCPP_INLINE_VISIBILITY size_t size() const {return __size_;}
_LIBCPP_INLINE_VISIBILITY size_t stride() const {return __stride_;}
+
+#if _LIBCPP_STD_VER >= 20
+
+ _LIBCPP_HIDE_FROM_ABI friend bool operator==(const slice& __x, const slice& __y) {
+ return __x.start() == __y.start() && __x.size() == __y.size() && __x.stride() == __y.stride();
+ }
+
+#endif
};
template <class _Tp> class _LIBCPP_TEMPLATE_VIS slice_array;
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D152617.530202.patch
Type: text/x-patch
Size: 2175 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/libcxx-commits/attachments/20230610/6f1d7ac3/attachment.bin>
More information about the libcxx-commits
mailing list