[PATCH] D83505: [NFC] Add utility to sum/merge stats files

Tyker via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 9 13:47:47 PDT 2020


Tyker added a comment.

In D83505#2142556 <https://reviews.llvm.org/D83505#2142556>, @lebedev.ri wrote:

> Is there really no such existing script? (i don't know of one, i've always hacked around with bash)


i tried to find one the first time i needed one and didn't find any. maybe there is one i didn't find.

> It's a non-blocker, but i just want to point out that aggregating via sum is not always correct for all stats - there's `updateMax()` setter.

ok
i wasn't aware of it, i don't think it is really possible to differentiate between them. but we could count both the sum and the max for all stats.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D83505





More information about the llvm-commits mailing list