[binutils-gdb] gprofng: fix sorting in Hist_data::sort

Vladimir Mezentsev vmezents@sourceware.org
Tue Dec 10 21:47:33 GMT 2024


https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=ded4c0a10e93235f19e967551a717d9348ebb1d9

commit ded4c0a10e93235f19e967551a717d9348ebb1d9
Author: Vladimir Mezentsev <vladimir.mezentsev@oracle.com>
Date:   Mon Dec 9 18:33:48 2024 -0800

    gprofng: fix sorting in Hist_data::sort
    
    If the '-name soname' option is used, the fake '<Total>' function is expanded
    with the name loadobject.
    
    gprofng/ChangeLog
    2024-12-09  Vladimir Mezentsev  <vladimir.mezentsev@oracle.com>
    
            * src/Hist_data.cc (Hist_data::sort): Fix sorting.

Diff:
---
 gprofng/src/Hist_data.cc | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/gprofng/src/Hist_data.cc b/gprofng/src/Hist_data.cc
index fb729018e0a..35d1f866a82 100644
--- a/gprofng/src/Hist_data.cc
+++ b/gprofng/src/Hist_data.cc
@@ -521,12 +521,11 @@ Hist_data::sort (long ind, bool reverse)
     hist_items->sort ((CompareFunc) sort_compare_all, this);
 
   // ensure that <Total> comes first/last
-  char *tname = NTXT ("<Total>");
   for (int i = 0; i < hist_items->size (); ++i)
     {
       HistItem *hi = hist_items->fetch (i);
       char *name = hi->obj->get_name ();
-      if (name != NULL && streq (name, tname))
+      if (name != NULL && strncmp (name, "<Total>", 7) == 0)
 	{
 	  int idx0 = rev_sort ? hist_items->size () - 1 : 0;
 	  if (i != idx0)


More information about the Binutils-cvs mailing list