[PATCH] D83942: [analyzer][tests] Add a notion of project sizes

2020-08-24 Thread Valeriy Savchenko via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGaec12c1264ac: [analyzer][tests] Add a notion of project sizes (authored by vsavchenko). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST AC

[PATCH] D83942: [analyzer][tests] Add a notion of project sizes

2020-08-07 Thread Artem Dergachev via Phabricator via cfe-commits
NoQ accepted this revision. NoQ added a comment. This revision is now accepted and ready to land. I can't say any of my ideas about better naming are exceptionally bright. The existing scale seems perfectly reasonable. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://r

[PATCH] D83942: [analyzer][tests] Add a notion of project sizes

2020-08-06 Thread Valeriy Savchenko via Phabricator via cfe-commits
vsavchenko updated this revision to Diff 283548. vsavchenko added a comment. Herald added a subscriber: steakhal. Keep SATest.py Python2 compatible Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D83942/new/ https://reviews.llvm.org/D83942 Files: c

[PATCH] D83942: [analyzer][tests] Add a notion of project sizes

2020-07-22 Thread Valeriy Savchenko via Phabricator via cfe-commits
vsavchenko updated this revision to Diff 279738. vsavchenko added a comment. Make --projects and --max-size compatible Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D83942/new/ https://reviews.llvm.org/D83942 Files: clang/utils/analyzer/ProjectMa

[PATCH] D83942: [analyzer][tests] Add a notion of project sizes

2020-07-20 Thread Valeriy Savchenko via Phabricator via cfe-commits
vsavchenko added a comment. In D83942#2156320 , @NoQ wrote: > > Sizes assigned to the projects in this commit, do not directly > > correspond to the number of lines or files in the project. > > Maybe `QUICK`/`NORMAL`/`SLOW` then? Or by purpose: > `BENCHM

[PATCH] D83942: [analyzer][tests] Add a notion of project sizes

2020-07-16 Thread Artem Dergachev via Phabricator via cfe-commits
NoQ added a comment. > Sizes assigned to the projects in this commit, do not directly > correspond to the number of lines or files in the project. Maybe `QUICK`/`NORMAL`/`SLOW` then? Or by purpose: `BENCHMARK`/`DAILY`/`PARANOID`? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTIO

[PATCH] D83942: [analyzer][tests] Add a notion of project sizes

2020-07-16 Thread Valeriy Savchenko via Phabricator via cfe-commits
vsavchenko marked an inline comment as done. vsavchenko added a comment. In 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!) CMa

[PATCH] D83942: [analyzer][tests] Add a notion of project sizes

2020-07-16 Thread Kristóf Umann via Phabricator via cfe-commits
Szelethus added a comment. I suppose LLVM could be a HUGE project? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D83942/new/ https://reviews.llvm.org/D83942 ___ cfe-commits mailing list cfe-commits@list

[PATCH] D83942: [analyzer][tests] Add a notion of project sizes

2020-07-16 Thread Kristóf Umann via Phabricator via cfe-commits
Szelethus added a comment. I don't speak snek, but I approve this message! Its also great that we have dedicated tiny/small projects to analyze locally. Comment at: clang/utils/analyzer/ProjectMap.py:31 +SMALL: 1min-10min +BIG: 10min-1h +HUGE: >1h --

[PATCH] D83942: [analyzer][tests] Add a notion of project sizes

2020-07-16 Thread Valeriy Savchenko via Phabricator via cfe-commits
vsavchenko created this revision. vsavchenko added reviewers: NoQ, xazax.hun, Szelethus, dcoughlin. Herald added subscribers: cfe-commits, ASDenysPetrov, Charusso, dkrupp, donat.nagy, mikhail.ramalho, a.sidorin, rnkovacs, szepet, baloghadamsoftware. Herald added a project: clang. Whith the number