Messages in this thread | | | From | Milian Wolff <> | Subject | Re: [PATCH v5 11/16] perf report: properly handle branch count in match_chain | Date | Mon, 16 Oct 2017 10:27:41 +0200 |
| |
On Montag, 16. Oktober 2017 06:18:17 CEST ravi wrote: > On Friday 13 October 2017 07:38 PM, Arnaldo Carvalho de Melo wrote: > > Em Fri, Oct 13, 2017 at 10:39:03AM -0300, Arnaldo Carvalho de Melo escreveu: > >> Em Mon, Oct 09, 2017 at 10:33:05PM +0200, Milian Wolff escreveu: > >>> Some of the code paths I introduced before returned too early > >>> without running the code to handle a node's branch count. > >>> By refactoring match_chain to only have one exit point, this > >>> can be remedied. > >> > >> Fixing up this one now. > > > > Millian, this is all fresher in your mind, can you please take a look at > > my perf/core branch and check if the change i made to ]PATCH v5 09/16] > > "perf report: compare symbol name for inlined frames when matching" is > > ok wrt Ravi's fix and then, please, rebase v5 on top of what is there? > > > > Ravi, please take a look at this as well, to see if with these changes > > your fix remains valid, ok? > > Yes Arnaldo, my changes are still valid. > > Milian, Can you please change this patch such that it incorporates dso > comparison for CCKEY_FUNCTION.
Arnaldo has already done that.
> ( Also, will that be good to change macro to CCKEY_FUNCTION_DOS ?)
Personally, I don't think so. The DSO compare is, imo, just an implementation detail.
Cheers
-- Milian Wolff | milian.wolff@kdab.com | Senior Software Engineer KDAB (Deutschland) GmbH&Co KG, a KDAB Group company Tel: +49-30-521325470 KDAB - The Qt Experts
| |