[libcxx-commits] [PATCH] D118029: Introduce branchless sorting functions for sort3, sort4 and sort5.

Nikolas Klauser via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Mon Jan 24 04:12:15 PST 2022


philnik requested changes to this revision.
philnik added a comment.
This revision now requires changes to proceed.

A few points right now:
(1) Please use `using x = `. It's a clang extension in C++03, so we can use it.
(2) `__magic_swap` is a really bad name. Please change that to something more meaningful.
(3) Don't qualify any type traits with `_VSTD::`.
(4) You can't use `enable_if_t`. Use `__enable_if_t` instead.
I'll have a closer look later.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D118029



More information about the libcxx-commits mailing list