Carlos =?utf-8?q?Gálvez?= <carlos.gal...@zenseact.com>, Carlos =?utf-8?q?Gálvez?= <carlos.gal...@zenseact.com> Message-ID: In-Reply-To: <llvm.org/llvm/llvm-project/pull/128...@github.com>
steakhal wrote: > That's a good point. @haoNoQ @Xazax-hun @steakhal Do you see any implications > on this patch when running clang static analyzer via clang-tidy? Any checks > that would stop detecting issues? > > TLDR: we are setting the traversal scope to only contain top-level > declarations which are not in system headers. Not analyzing system headers is the desired behavior for the Static Analyzer too. But please note that the `clang/lib/StaticAnalyzer/Frontend/AnalysisConsumer.cpp` does it's own `HandleTopLevelDecl` thing, thus the Static Analyzer wouldn't be affected by setting this scope here. That code would need to be patched separately to somehow take this into account. If we would patch it, we would of course lose issues, but those were not actionable anyway. So, it would be an improvement in that sense. I strongly support the direction. https://github.com/llvm/llvm-project/pull/128150 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
