[PATCH] D53978: Add benchmarks for sorting and heap functions.

Samuel Benzaquen via Phabricator reviews at reviews.llvm.org
Thu Nov 1 09:08:52 PDT 2018


sbenza added a comment.

In https://reviews.llvm.org/D53978#1283922, @mclow.lists wrote:

> Is there any value in breaking these out into one benchmark per file? I can see this file ever-growing as we add new benchmarks.


No idea. I was just following the pattern of one benchmark .cpp per header. I think <algorithm> would be the only one with the potential bloat.


Repository:
  rCXX libc++

https://reviews.llvm.org/D53978





More information about the libcxx-commits mailing list