Alexander_Droste added a comment. > I'd be fine if we test this function with the usual regression tests by > observing the output of the MPI checker. We could update that test with more > checks once the function is updated.
> With that approach, you'd be committing both patches at the same time. You mean to: Temporarily remove the `MemRegion.cpp` testfile -> committing `getDescriptiveName` -> committing `sourceRange` -> committing MPI-Checker -> then readding the testfile; right? http://reviews.llvm.org/D16044 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
