[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 15:35:36 PST 2015


silvas added a comment.

In http://reviews.llvm.org/D15306#307692, @davidxl wrote:

> 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?


Yes. What I'm suggesting is precisely like the current patch, except that `--weighted-file=<weight>,<filename>` is used instead of `<filename>:<weight>`. (I do not care about `--weighted-file` syntax per se; it could be `-weighted-file`, `-file-with-weight`, etc. whatever is most convenient to implement. The fundamental thing is to reuse the user intuition about passing options in order to avoid exposing users to another special case (argument syntax is already a special case, but one that users are aware of).


http://reviews.llvm.org/D15306





More information about the llvm-commits mailing list