[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