[libcxx-commits] [PATCH] D63063: Bug 42208: speeding up std::merge

[Denis Yaroshevskiy via Phabricator via libcxx-commits]

dyaroshev updated this revision to Diff 212127.
dyaroshev added a comment.

Eric's comment.

I don't know - I had to modify some changes from: https://github.com/llvm-mirror/libcxx/commit/c005c7e34c3d22d1dba2cfe62c79f9e8be2d60de

Why are those 'constexpr if nodebug' and why does it make sense?

Could not find the 'phabricator review' or anything.
Alternatively - I can disable the mmemove optimization in debug completely - what do you think about that?


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

https://reviews.llvm.org/D63063

Files:
  benchmarks/algorithms.merge.bench.cpp
  include/algorithm
  include/iterator
  test/libcxx/algorithms/merge.pass.cpp
  test/std/algorithms/alg.modifying.operations/alg.copy/copy.pass.cpp
  test/std/algorithms/alg.sorting/alg.merge/merge.pass.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D63063.212127.patch
Type: text/x-patch
Size: 19360 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/libcxx-commits/attachments/20190728/91ff5da6/attachment-0001.bin>