- User Since
- Aug 20 2015, 4:19 PM (209 w, 22 h)
Fri, Aug 2
Thu, Aug 1
Mon, Jul 29
Integrated Chandler's review comments.
I'm sorry that I missed this review for this long!
Jul 9 2019
Jul 8 2019
Change the error report format suggested by David.
Jul 3 2019
Jul 1 2019
Sent the wrong test file in last patch.
Update to the correct one.
Jun 28 2019
they are not doing the exacly the same thing for old pass manager and new pass manger: old pass manger is doing instrumentation, while the new pass manager with this change is NOT.
the test is not check instrumentation, (it only check the variables that generates by InstroProfiling pass).
In this sense, the test is not well written.
This patch does not make sense to me.
Jun 10 2019
Integrated David's review suggestions.
Jun 7 2019
Jun 3 2019
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.
May 30 2019
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.
May 28 2019
Fix comments suggested by David.
Integrated comments from Noel and David.
May 24 2019
May 20 2019
May 3 2019
Apr 30 2019
Apr 29 2019
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
Apr 26 2019
Integrated David's review comments. Mainly changed the reporting format.
Apr 25 2019
Apr 24 2019
Apr 23 2019
Integrated David's review comments.
LGTM. We need to Initialize the OptLevel no matter what.
Apr 22 2019
Removed some un-intended changes.
Minor fix of the comments.
Apr 18 2019
Apr 11 2019
Apr 10 2019
Integrated David's comments.
Apr 2 2019
Mar 12 2019
Mar 11 2019
Integrated Ruiu's review comments.
Mar 8 2019
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.