Re: [PATCH v6 0/7] perf report: Show branch type

From: Jin, Yao
Date: Sun Apr 23 2017 - 04:41:35 EST




On 4/20/2017 5:36 PM, Jiri Olsa wrote:
On Thu, Apr 20, 2017 at 08:07:48PM +0800, Jin Yao wrote:
v6:
Update according to the review comments from
Jiri Olsa <jolsa@xxxxxxxxxx>. Major modifications are:

1. Move that multiline conditional code inside {} brackets.

2. Move branch_type_stat_display() from builtin-report.c to
branch.c. Move branch_type_str() from callchain.c to
branch.c.

3. Keep the original branch info display order, that is:
predicted, abort, cycles, iterations
for the tools part

Acked-by: Jiri Olsa <jolsa@xxxxxxxxxx>

thanks,
jirka

Thanks so much!

Is the kernel part OK?

Thanks
Jin Yao