Alexander_Droste added inline comments. ================ Comment at: test/Analysis/MPIChecker.cpp:99 @@ +98,3 @@ + MPI_Wait(&req, MPI_STATUS_IGNORE); +} + ---------------- Alexander_Droste wrote: > zaks.anna wrote: > > This are explaining the path on which the problem occurs; the users will > > see them as well. There should not be a lot of those, you do not have a lot > > of conditions. Would it be reasonable to change the tests to incorporate > > those. Other alternative is to have another tests file that tests the notes > > in that mode. > > > > What do you think? > I'm fine with adding the notes to this test file. I had a look once more how much output this actually produces. If `-analyzer-output=text` is passed, 42 notes are missing. I'll add another test file which will test for these notes.
http://reviews.llvm.org/D12761 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
