[libc-commits] [PATCH] D120040: [libc][automemcpy] Introduce geomean of scores as a tie breaker

Clement Courbet via Phabricator via libc-commits libc-commits at lists.llvm.org
Thu Feb 17 04:33:53 PST 2022


courbet accepted this revision.
courbet added inline comments.
This revision is now accepted and ready to land.


================
Comment at: libc/benchmarks/automemcpy/lib/ResultAnalyzerMain.cpp:131-133
+              return A.FinalGrade == B.FinalGrade
+                         ? (A.ScoresGeoMean > B.ScoresGeoMean)
+                         : (A.FinalGrade < B.FinalGrade);
----------------
use the `tie` trick ? (`make_tuple` in this case since you have values) 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D120040



More information about the libc-commits mailing list