Integrated David's review suggestions.

In D62700#1536841, @davidxl wrote:

This can be handled this way:

BBInfo &SrcInfo = getBBInfo(SrcBB);    // BBInfo is the template type
BBInfo &DestInfo = getBBInfo(DestBB);
SrcInfo.addOutEdge(E.get());
DestInfo.addInEdge(E.get());

For base class BBInfo, addOutEdge, addInEdge will be empty, while it is defined for UseBBInfo class.

By so doing, the code is more readable.

My previous patch was not complete -- we need to set the profile count to the fail-to-instrument edge. Otherwise, we cannot propagate the counts in the CFG.
The new patch has this part of change.
Updated the test to test the profile-use.

Another change of this patch is to move down statistic collection code for critical edge. I think this is a better place.

looks good to me.

Fix comments suggested by David.

Integrated comments from Noel and David.

Adjusted the output string with David's comments.
Also made the naming more uniformed for edge and value profiles.

Discussed offline with David. We changed the output to a simpler: "Description: <Value>" format. This tool will mostly used in a script. Simpler format will make text grep cleaner.

Integrated David's review comments

Integrated David's review comments. Mainly changed the reporting format.

Integrated David's review comments.

LGTM. We need to Initialize the OptLevel no matter what.

Removed some un-intended changes.

Minor fix of the comments.

Integrated David's comments.

In D56675#1426458, @dmajor wrote:

Are there any bugs/reviews/etc. that I can subscribe to for the implementation of this in COFF?

Integrated Ruiu's review comments.

Ping.

using utils/update_test_checks.py to generate the test checks.

changed test case based on spatel's comments.

add one missing updates in last version of the patch.

Integrated Teresa's comments.