[PATCH] D14547: [llvm-profdata] Add support for weighted merge of profile data

David Li via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 10 11:39:04 PST 2015


davidxl added a comment.

Thanks -- this is a good functionality.

A high level comment about the command line syntax. I think we can do it without introducing a new option. All that needed for user is to add an optional weight suffix to the input file name such as:

llvm-profdata  merge -o outp.profdata  input1.data:10 input2.data:5 input3.data

When the :weight suffix is missing, the weight is assumed to be 1.

Note that this will like to simplify the implementation quite a bit -- not special parsing is needed.


http://reviews.llvm.org/D14547





More information about the llvm-commits mailing list