[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
Tue Dec 8 14:50:47 PST 2015


The subtlety is pretty well understood and the solution to it is clean
-- I don't see a strong reason to introduce a new 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