[libcxx-commits] [PATCH] D142183: [libc++] Split ranges.transform.binary.pass.cpp up

[Mark de Wever via Phabricator via libcxx-commits]

Mordante added a comment.

Thanks for working on this!
On AIX 64 both tests now take about the same time.
555.11s: ibm-libc++-shared.cfg.in :: std/algorithms/alg.modifying.operations/alg.transform/ranges.transform.binary.range.pass.cpp
530.22s: ibm-libc++-shared.cfg.in :: std/algorithms/alg.modifying.operations/alg.transform/ranges.transform.binary.iterator.pass.cpp



================
Comment at: libcxx/test/std/algorithms/alg.modifying.operations/alg.transform/ranges.transform.binary.iterator.pass.cpp:20
+//     ranges::transform(I1 first1, S1 last1, I2 first2, S2 last2, O result,
+//                       F binary_op, Proj1 proj1 = {}, Proj2 proj2 = {});
+
----------------
Please add a remark `The ranges part is tested in ranges.transform.binary.ranges.pass.cpp`, and a similar in the other file.



Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D142183/new/

https://reviews.llvm.org/D142183