[libcxx-commits] [PATCH] D122173: [libc++][ranges] Implement ranges::transform

[Nikolas Klauser via Phabricator via libcxx-commits]

philnik added inline comments.


================
Comment at: libcxx/test/std/algorithms/alg.modifying.operations/alg.transform/ranges.transform.pass.cpp:408
+
+  { // check that std::ranges::dangling is returned properly
+    { // unary
----------------
ldionne wrote:
> Would it be possible to move these tests into the `test_iterators` function? LMK if compile-times explode in a bad way.
The test takes 40s to compile and run on my system now. The tests that aren't in `test_iterators` can't be moved there easily, because they don't use `int`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D122173