[PATCH] D15306: [llvm-profdata] Add support for weighted merge of profile data (2nd try)

[Xinliang David Li via llvm-commits]

sounds good to me.

David

On Thu, Dec 10, 2015 at 4:02 PM, Sean Silva <chisophugis at gmail.com> wrote:
> silvas added a comment.
>
> In http://reviews.llvm.org/D15306#307704, @slingn wrote:
>
>> There's one --weighted-file=<weight>,<filename> per file to merge, right?
>
>
> Sorry, missed this in my last message. Yes. Any `<filename>` can be replaced with `--weighted-file=<weight>,<filename>`. Just `<filename>` is equivalent to `--weighted-file=1,<filename>`.
>
> A couple examples are the easiest way to explain it (especially to users) I think:
>
> (something like this would be useful to include in the docs I think)
>
>   # Basic usage
>   llvm-profdata merge foo.profdata bar.profdata baz.profdata
>
>   # `foo.profdata` is especially important, multiply its counts by 10.
>   llvm-profdata merge --file-with-weight=10,foo.profdata bar.profdata baz.profdata
>   # Exactly equivalent to the previous invocation (explicit form; useful for programmatic invocation)
>   llvm-profdata merge --file-with-weight=10,foo.profdata --file-with-weight=1,bar.profdata --file-with-weight=1,baz.profdata
>
>
> http://reviews.llvm.org/D15306
>
>
>