[PATCH] D54388: [llvm-exegesis] InstructionBenchmarkClustering::rangeQuery(): use llvm::SmallVector<size_t, 0> for storage.

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 19 04:00:51 PST 2018


lebedev.ri added a comment.

In https://reviews.llvm.org/D54388#1302661, @courbet wrote:

> You don't need this one anymore right ? Though I don't oppose to you keeping it for benchmarking reference.


Correct. It has been effectively superseeded by further patches, but it would make sense to keep it for consistency, as a fine-grained refactoring step.
Thank you for the review!


Repository:
  rL LLVM

https://reviews.llvm.org/D54388





More information about the llvm-commits mailing list