[libcxx-commits] [PATCH] D149827: [libc++][ranges] Implement the changes to `deque` from P1206 (`ranges::to`):

[Konstantin Varlamov via Phabricator via libcxx-commits]

var-const added a comment.

A note in case it every comes in handy later: while working on this patch, I noticed that `deque` tests are significantly (5-10x) slower compared to the equivalent `vector` tests when compiled by GCC; with Clang, the difference is almost negligible. This is a preexisting issue -- running the test suite for `insert_range` while using the old `insert` function that takes two iterators on `main` shows a similarly large performance difference when compiling with GCC. From a conversation with @ldionne, one explanation could be that when compiling under GCC, many functions get force-inlined which might affect `deque` more significantly (and negatively) than `vector`. In any case, this patch doesn't make the situation worse.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D149827