[PATCH] D52390: [analyzer] StackSizeChecker

[Roman Lebedev via Phabricator via cfe-commits]

lebedev.ri added a comment.

In https://reviews.llvm.org/D52390#1243228, @mate1214 wrote:

> Hi @lebedev.ri!
>
> Thanks for the question. I was not sure as to where exactly put the files. The important thing for me is that the `StackUsageMeasuringVisitor` should be reachable from the clang-tidy checker. (For the bigger picture please refer to my answer to @Szelethus). I was not able to find documentation I could use to put the extra logic in the right place (may be my fault). Any suggestions and references are welcome!


Reachable *how*?
Is clang-tidy check using this very same logic?
For recent examples, see ExprMutationAnalyzer, how it is structured (especially the tests, they are **so** nice!).
It started in clang-tidy, but then was promoted to analysis/.
So i'm wondering if the same should happen here, too.


Repository:
  rC Clang

https://reviews.llvm.org/D52390