[PATCH] D54389: [llvm-exegesis] InstructionBenchmarkClustering::dbScan(): replace std::vector<> with std::deque<> in llvm::SetVector<>

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

Previous message: [PATCH] D54383: [llvm-exegesis] Analysis: writeMeasurementValue(): don't alloc string for double each time.
Next message: [PATCH] D54389: [llvm-exegesis] InstructionBenchmarkClustering::dbScan(): replace std::vector<> with std::deque<> in llvm::SetVector<>
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

lebedev.ri added a comment.

In https://reviews.llvm.org/D54389#1296023, @RKSimon wrote:

> LGTM as part of the patch series


@courbet

Thank you for the reviews!


Repository:
  rL LLVM

https://reviews.llvm.org/D54389

Previous message: [PATCH] D54383: [llvm-exegesis] Analysis: writeMeasurementValue(): don't alloc string for double each time.
Next message: [PATCH] D54389: [llvm-exegesis] InstructionBenchmarkClustering::dbScan(): replace std::vector<> with std::deque<> in llvm::SetVector<>
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

More information about the llvm-commits mailing list