[libc-commits] [PATCH] D111624: [libc] automemcpy - result analyzer

Clement Courbet via Phabricator via libc-commits libc-commits at lists.llvm.org
Wed Oct 20 02:51:06 PDT 2021


courbet added a comment.

This could really use some doc.



================
Comment at: libc/benchmarks/automemcpy/lib/ResultAnalyzer.cpp:160-162
+  StringMap<double> Throughputs; // Median of samples per distribution
+  StringMap<double> Scores;      // Normalized score per distribution
+  StringMap<Grade::GradeEnum> GradeAttribution; // Grade per distribution
----------------
any reason not to group `{Throughput,Score,GradeAttribution}` to avoid the 3 parallel maps ?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D111624/new/

https://reviews.llvm.org/D111624



More information about the libc-commits mailing list