I have one comment: "valgrind" sounds like a very tool-specific option.<br>Are there other memory checking tools?<br>Maybe there should be an option like "-memcheck [valgrind]" with an optional tool name,<br>
similar to "-start_in_debugger [gdb]"?<br><br>Dmitry.<br><br><div class="gmail_quote">On Fri, Aug 21, 2009 at 4:44 PM, Barry Smith <span dir="ltr"><<a href="mailto:bsmith@mcs.anl.gov">bsmith@mcs.anl.gov</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><br>
   I noticed valgrind commands creeping into PETSc example makefiles. I believe this is a terrible model and would like to remove them.<br>
<br>
   I've added support for calling valgrind correctly from linux or the Mac to $PETSC_DIR/bin/petscmpiexec as an optional first argument -valgrind.<br>
Thus you can call any PETSc program with valgrind using, for example, petscmpiexec -valgrind -n 2 ./ex1 -ksp_monitor<br>
<br>
   Now for makefile tests one can run, for example: make runex7 MPEXEC="petscmpiexec -valgrind" and it will run the runex7 test under valgrind.<br>
Is this an acceptable replacement for having all the horrible redundant use of valgrind directly in the makefiles and do I have everyones permission to remove the use of valgrind from the makefiles? If not, what support do I need to add to get rid of the use of valgrind from the makefiles?<br>
<font color="#888888">
<br>
<br>
   Barry<br>
</font></blockquote></div><br>