[PATCH] D83942: [analyzer][tests] Add a notion of project sizes
Valeriy Savchenko via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Jul 16 07:24:44 PDT 2020
vsavchenko marked an inline comment as done.
vsavchenko added a comment.
In D83942#2155762 <https://reviews.llvm.org/D83942#2155762>, @Szelethus wrote:
> I don't speak snek, but I approve this message!
Thanks 😊
> I suppose LLVM could be a HUGE project?
Yurp, LLVM, pytorch, (surprisingly!) CMake
================
Comment at: clang/utils/analyzer/ProjectMap.py:31
+ SMALL: 1min-10min
+ BIG: 10min-1h
+ HUGE: >1h
----------------
Szelethus wrote:
> Just an observation rather then a suggestion, its interesting that we don't have a MEDIUM size in between SMALL and BIG. I think the TINY category describes the sub-minute runs well, and it'd be awkward to introduce a project in between SMALL and BIG, so I don't immediately see a time interval we need to categorize.
Yeah, maybe it's good to add MEDIUM with 10min-30min
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D83942/new/
https://reviews.llvm.org/D83942
More information about the cfe-commits
mailing list