[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