[PATCH] D25985: [analyzer] Export coverage information from the analyzer.
Anna Zaks via cfe-commits
cfe-commits at lists.llvm.org
Wed Nov 9 16:33:22 PST 2016
zaks.anna added a comment.
> Added a python script to merge gcov files.
When the patches that are being reviewed are growing new functionality, it's much more difficult to review them. Please, submit the python script as a separate patch.
================
Comment at: lib/StaticAnalyzer/Core/ExprEngine.cpp:262
+
+static void dumpCoverageInfo(llvm::SmallVectorImpl<char> &Path,
+ SourceManager &SM) {
----------------
xazax.hun wrote:
> zaks.anna wrote:
> > Can this be a debug checker?
> You mean the dumping part or also collecting the coverage? It can be a debug checker if we add a new callback like "checkBasicBlockBegin". I did not want to add such callback just for a debug check. However, it is possible to move only the dumping part to a separate checker. What would you prefer?
I would prefer to add a "checkBasicBlockBegin" callback and move all of this diagnostic code into a debug checker.
================
Comment at: lib/StaticAnalyzer/Core/ExprEngine.cpp:277
+ if (!OutFile) {
+ llvm::errs() << FilePath << " Fuck!\n";
+ continue;
----------------
zaks.anna wrote:
> Please, come up with constructive error messages.
How was this addressed?
https://reviews.llvm.org/D25985
More information about the cfe-commits
mailing list