[libcxx-commits] [PATCH] D110495: [libc++][format][2/6] Adds a __output_iterator.

[Louis Dionne via Phabricator via libcxx-commits]

ldionne added a comment.

Sorry, I hadn't seen this was waiting for some sort of resolution.

@Mordante do you have an already planned way of addressing the inefficiencies raised by @vitaut? If not, I'd like to have a solution for those before shipping this. Otherwise, there's a risk that we'll forget about it and eventually ship the library without these optimizations (which IIUC are not merely nice-to-haves, but really important basic guarantees). I especially want to avoid us being painted into a corner where these optimizations are difficult to achieve in the current design.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D110495