[libcxx-commits] [PATCH] D122074: [libc++] Allow an output_iterator in fill.

[Mark de Wever via Phabricator via libcxx-commits]

Mordante added a comment.

In D122074#3394314 <https://reviews.llvm.org/D122074#3394314>, @philnik wrote:

> Would `ranges::fill_n` work for your purposes? If yes, maybe just use that (I can work on implementing it) instead of adding an extension?

Good point, when I initially ran into this issue ranges was quite less mature so I never even considered it. But it should be possible.
If you want to look into it that would be great! From a quick look at the format code I use `copy`, `copy_n`, `fill_n`, and `transform`.

The `copy` and `copy_n` are a bit tricky due to the `wrap_iterator`.

@ldionne Do you see any value in this extension or do you prefer to abandon this patch?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D122074