[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