[PATCH] Initial code coverage mapping data structures, and reader and writers + C interface for ProfileData library

Alex L arphaman at gmail.com
Thu Jun 26 09:46:19 PDT 2014


2014-06-25 18:19 GMT-07:00 Justin Bogner <mail at justinbogner.com>:

> Alex L <arphaman at gmail.com> writes:
> > There is a link to phabricator or is it not visible?
>
> The canonical place for code reviews is on this list, so the patch
> should be sent here.
>

I've attached the patch with this email.


>
> I often see phabricator sending the patch to the list when you create a
> review - from [1] it looks like this is done by adding llvm-commits as a
> reviewer. This may also be needed for comments on phabricator to come to
> the list, which is important so that developers that don't use
> phabricator can participate and see what's happening.
>
> [1]: http://llvm.org/docs/Phabricator.html
>

Thanks, I didn't add llvm-commits yesterday, I will do it now.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140626/a9c67c40/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: coverageFormat.patch
Type: application/octet-stream
Size: 53153 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140626/a9c67c40/attachment.obj>


More information about the llvm-commits mailing list