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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to