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

Sean Silva via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 10 16:02:00 PST 2015


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





More information about the llvm-commits mailing list