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

[Sean Silva via llvm-commits]

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?
>
> Regarding intuition - I know I have trouble remember the exact name of options I don't use very often. I might have a chance with a single ':'. :)


That is not intuition; that is memorableness. The reality is that it is unlikely that a user will type in the merge step by hand except maybe to try it out once or twice at first (in which case it's all new to them anyway).

It is more important that once a user's automation has been taught to invoke the merge step, that it robustly continues to work as intended. Also, it is important for us to be able to easily communicate to them how to robustly invoke the tool.


http://reviews.llvm.org/D15306