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

Xinliang David Li via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 10 15:24:22 PST 2015


I misunderstood your proposal. Your proposal is a modified version of
the originally proposed option that requires weight to be specified
for every file (and with implicit ordering assumption).  To specify
more than one file with weight, does user have to use multiple
--weight-file option?

David

On Tue, Dec 8, 2015 at 2:38 PM, Sean Silva <chisophugis at gmail.com> wrote:
> silvas added a subscriber: silvas.
> silvas added a comment.
>
> Given the subtleties we've encountered, I propose we switch over to using the syntax `--weighted-file=<weight>,<filename>` (or similar). This eliminates the special cases and is easier to document without subtly lying about the true behavior.
>
>
> http://reviews.llvm.org/D15306
>
>
>


More information about the llvm-commits mailing list