[libcxx-commits] [PATCH] D92190: [libc++] fix std::sort(T**, T**)

Louis Dionne via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Tue Dec 8 10:29:05 PST 2020


ldionne added inline comments.


================
Comment at: libcxx/include/algorithm:4163
 void
 sort(_Tp** __first, _Tp** __last)
 {
----------------
@howard.hinnant 

Sorry I wasn't clear when I pinged you earlier -- but do you recall why you wrote this specialization of `std::sort` for ranges containing pointers? Was it to avoid instantiating the actual sort algorithm on various `T*` types and save on code size?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D92190



More information about the libcxx-commits mailing list