[PATCH] Added basic testing support for llvm-cov.

Yuchen Wu yuchenericwu at hotmail.com
Fri Nov 1 21:12:48 PDT 2013


Updated. 

If this looks okay, I'll commit it after the patch for reading run counts gets the LGTM, as this depends on it to not fail.

----------------------------------------
> From: mail at justinbogner.com
> To: yuchenericwu at hotmail.com
> CC: llvm-commits at cs.uiuc.edu
> Subject: Re: [PATCH] Added basic testing support for llvm-cov.
> Date: Fri, 1 Nov 2013 16:06:06 -0700
>
> Yuchen Wu <yuchenericwu at hotmail.com> writes:
>> Updated patch to avoid creating temporaries when testing
>> llvm-cov. Added new command-line option into llvm-cov that allows for
>> specifying the input file directory.
>
> Probably best to split this into two - one patch to add the command line
> option and another that adds the test. It'd also probably be cleaner to
> make the command line option more like make's "-C", where it just
> changes the working directory before doing anything. This would avoid
> the relative vs. absolute path logic.
>
> It'd also be nice to mention somewhere where the reference output you're
> comparing against came from (gcov version, for example). 		 	   		  
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0012-llvm-cov-Added-command-line-option-to-change-dir.patch
Type: application/octet-stream
Size: 3145 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20131102/8054137f/attachment.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0013-Added-basic-unit-test-for-llvm-cov.patch
Type: application/octet-stream
Size: 8129 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20131102/8054137f/attachment-0001.obj>


More information about the llvm-commits mailing list