[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