[libcxx-commits] [PATCH] D100876: [libc++] [test] Remove epicyclic workarounds for vector/span; use T[] or std::array

Arthur O'Dwyer via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Tue Apr 20 11:42:21 PDT 2021


Quuxplusone created this revision.
Quuxplusone added reviewers: ldionne, zoecarver, Mordante, curdeius.
Quuxplusone added a project: libc++.
Quuxplusone requested review of this revision.
Herald added a subscriber: libcxx-commits.
Herald added a reviewer: libc++.

Simplify the test code, and drive-by also test that these algorithms return the right iterator as their return value.

This came, obliquely, out of D100866 <https://reviews.llvm.org/D100866>. Fixing this test now will remove the dependency on `<span>`, which means I won't have to mark it XFAILed in debug mode because it no longer fails.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D100876

Files:
  libcxx/test/std/numerics/numeric.ops/exclusive.scan/exclusive_scan.pass.cpp
  libcxx/test/std/numerics/numeric.ops/exclusive.scan/exclusive_scan_init_op.pass.cpp
  libcxx/test/std/numerics/numeric.ops/inclusive.scan/inclusive_scan.pass.cpp
  libcxx/test/std/numerics/numeric.ops/inclusive.scan/inclusive_scan_op.pass.cpp
  libcxx/test/std/numerics/numeric.ops/inclusive.scan/inclusive_scan_op_init.pass.cpp
  libcxx/test/std/numerics/numeric.ops/transform.exclusive.scan/transform_exclusive_scan_init_bop_uop.pass.cpp
  libcxx/test/std/numerics/numeric.ops/transform.inclusive.scan/transform_inclusive_scan_bop_uop.pass.cpp
  libcxx/test/std/numerics/numeric.ops/transform.inclusive.scan/transform_inclusive_scan_bop_uop_init.pass.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D100876.338942.patch
Type: text/x-patch
Size: 35803 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/libcxx-commits/attachments/20210420/ea137f28/attachment-0001.bin>


More information about the libcxx-commits mailing list