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

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 9 13:50:10 PDT 2020


lebedev.ri added a comment.

In D83505#2142581 <https://reviews.llvm.org/D83505#2142581>, @Tyker wrote:

> 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.


Yep. Just feel like pointing out.

> 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