[PATCH] D16044: getDescriptiveName() for MemRegion

[Alexander Droste via cfe-commits]

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