[PATCH] D14094: Add multikey_qsort to STLExtras.h

David Blaikie via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 26 15:29:49 PDT 2015


On Mon, Oct 26, 2015 at 3:26 PM, Sean Silva via llvm-commits <
llvm-commits at lists.llvm.org> wrote:

> silvas added a subscriber: silvas.
> silvas added a comment.
>
> I'd really like to see sanity check performance numbers vs. std::sort to
> make sure that this function is worth it.
>

+1 - wouldn't mind seeing some more data on the options here. Not that this
piece of code is terribly costly to add & I can totally understand why a
linker would need a sort algorithm specifically designed to cope with
inputs with large common prefixes.


>
>
> http://reviews.llvm.org/D14094
>
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20151026/bcb065b3/attachment.html>


More information about the llvm-commits mailing list