[cfe-dev] add a valgrind option to "make test"

Sam Bishop sam at bishop.dhs.org
Sun Mar 16 14:25:34 PDT 2008


Attached is a patch which adds a valgrind option to "make test".  The
patch can be merged as-is, but there are a few things I'm unsure about:

o  Why the original "rm $OUTPUT" if something goes wrong?
o  Right now I replace "clang" with "valgrind ... clang".  Should we
introduce a "%clang" instead?  That has pros and cons.

Also, I haven't touched Makefile.parallel.  That should be trivial, though.

Thanks,
Sam
-------------- next part --------------
A non-text attachment was scrubbed...
Name: valgrind-make-test.patch
Type: application/octet-stream
Size: 1842 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-dev/attachments/20080316/3a37b138/attachment.obj>


More information about the cfe-dev mailing list