[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