[PATCH] D52390: [analyzer] Add StackSizeChecker to StaticAnalyzer

2018-10-16 Thread Máté Tóth via Phabricator via cfe-commits
mate1214 abandoned this revision. mate1214 marked 24 inline comments as done. mate1214 added a comment. Herald added subscribers: dkrupp, nhaehnle, donat.nagy. Hi! Sorry for the delay. After considering the amount of requested tests and investigating the possibly reviewer friendly ways to split

[PATCH] D52390: [analyzer] StackSizeChecker

2018-09-24 Thread Máté Tóth via Phabricator via cfe-commits
mate1214 added a comment. 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 @Szelet

[PATCH] D52390: [analyzer] StackSizeChecker

2018-09-24 Thread Máté Tóth via Phabricator via cfe-commits
mate1214 added a comment. Hi @Szelethus ! Thanks for all your detailed and helpful input, I will make sure to go over all the comments and answer them, but it will take some time. A bit more background information on this checker and how it came to be might help you and others to understand so

[PATCH] D52390: [analyzer] StackSizeChecker

2018-09-22 Thread Máté Tóth via Phabricator via cfe-commits
mate1214 created this revision. mate1214 added reviewers: NoQ, george.karpenkov, dcoughlin. Herald added subscribers: cfe-commits, Szelethus, mikhail.ramalho, a.sidorin, szepet, mgorny. Add StackSizeChecker to StaticAnalyzer This checker can be used to warn about potential stack overflows