[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