* core.c (core_create_function_syms): Move filename and func_name

	inside ifdef where they are used.

	* core.c (core_sym_class): Parenthesize && within ||.
	* symtab.c (symtab_finalize): Correct parenthesization.

	* cg_print.h (cg_print_file_ordering): Declare.
	(cg_print_function_ordering): Declare.

	* __bb_exit_func.c (__bb_exit_func): Replace bcopy with memcpy.
	* cg_arcs.c (arc_add): Likewise.
	* cg_print.c (cg_print_function_ordering): Likewise.

diff --git a/gprof/ChangeLog b/gprof/ChangeLog
index 0cee8528817f44514cc2846c22a8f676a439d624..8fd3b562147e25c12424c453da3839f6c3299366 100644 (file)
--- a/gprof/ChangeLog
+++ b/gprof/ChangeLog
@@ -1,3 +1,18 @@
+Thu Mar 21 17:18:25 1996  Ian Lance Taylor  <ian@cygnus.com>
+
+       * core.c (core_create_function_syms): Move filename and func_name
+       inside ifdef where they are used.
+
+       * core.c (core_sym_class): Parenthesize && within ||.
+       * symtab.c (symtab_finalize): Correct parenthesization.
+
+       * cg_print.h (cg_print_file_ordering): Declare.
+       (cg_print_function_ordering): Declare.
+
+       * __bb_exit_func.c (__bb_exit_func): Replace bcopy with memcpy.
+       * cg_arcs.c (arc_add): Likewise.
+       * cg_print.c (cg_print_function_ordering): Likewise.
+
 Thu Mar 21 17:02:02 1996  David Mosberger-Tang  <davidm@azstarnet.com>
 
        * gprof.c (default_excluded_list): Add "__mcount".

diff --git a/gprof/__bb_exit_func.c b/gprof/__bb_exit_func.c
index 512056ed4af0fd25d51645c39ca02337faef7c68..813321566c72c1b59b3ce5dc652f38b6e008fdf9 100644 (file)
--- a/gprof/__bb_exit_func.c
+++ b/gprof/__bb_exit_func.c
@@ -54,8 +54,8 @@ __bb_exit_func (void)
        perror(OUT_NAME);
        return;
     } /* if */
-    bcopy(GMON_MAGIC, &ghdr.cookie[0], 4);
-    bcopy(&version, &ghdr.version, sizeof(version));
+    memcpy(&ghdr.cookie[0], GMON_MAGIC, 4);
+    memcpy(&ghdr.version, &version, sizeof(version));
     fwrite(&ghdr, sizeof(ghdr), 1, fp);
 
     for (ptr = __bb_head; ptr != 0; ptr = ptr->next) {

diff --git a/gprof/cg_arcs.c b/gprof/cg_arcs.c
index c6431e4b2eda787e603714c5aecd953de694e2df..b63102948b036f5cf9534ad16f958458d78b3e84 100644 (file)
--- a/gprof/cg_arcs.c
+++ b/gprof/cg_arcs.c
@@ -106,7 +106,7 @@ DEFUN (arc_add, (parent, child, count),
          newarcs = (Arc **)xmalloc(sizeof (Arc *) * maxarcs);
 
          /* Copy the old array's contents into the new array.  */
-         bcopy (arcs, newarcs, numarcs * sizeof (Arc *));
+         memcpy (newarcs, arcs, numarcs * sizeof (Arc *));
 
          /* Free up the old array.  */
          free (arcs);

diff --git a/gprof/cg_print.c b/gprof/cg_print.c
index 236e01b11fa0238f5d2a6389171deb0d27f11e49..015f710ce3229bd56f466629e7ffe9938c6f5abf 100644 (file)
--- a/gprof/cg_print.c
+++ b/gprof/cg_print.c
@@ -844,7 +844,7 @@ DEFUN_VOID (cg_print_function_ordering)
 
   /* Now sort a temporary symbol table based on the number of
      times each function was used in the highest used arcs.  */
-  bcopy (used_syms, scratch_syms, used * sizeof (Sym *));
+  memcpy (scratch_syms, used_syms, used * sizeof (Sym *));
   qsort (scratch_syms, used, sizeof (Sym *), cmp_fun_nuses);
 
   /* Now pick out those symbols we're going to emit as

diff --git a/gprof/cg_print.h b/gprof/cg_print.h
index 13511c728a024df3aa9ca6ef70713bcdd046b7e8..782c4aa04c26e38d2b00d73eb868e4227a77c80b 100644 (file)
--- a/gprof/cg_print.h
+++ b/gprof/cg_print.h
@@ -8,5 +8,7 @@ extern double print_time;       /* total of time being printed */
 
 extern void cg_print PARAMS ((Sym ** cg));
 extern void cg_print_index PARAMS ((void));
+extern void cg_print_file_ordering PARAMS ((void));
+extern void cg_print_function_ordering PARAMS ((void));
 
 #endif /* cg_print_h */

diff --git a/gprof/core.c b/gprof/core.c
index b54c86e3ba94336898fa0facdc0e7c93e1f37431..3117599eba0807ffee8c6a68bfc1fb73cf568981 100644 (file)
--- a/gprof/core.c
+++ b/gprof/core.c
@@ -270,7 +270,7 @@ DEFUN (core_sym_class, (sym), asymbol * sym)
    * other systems.  Perhaps it should be made configurable.
    */
   sym_prefix = bfd_get_symbol_leading_char (core_bfd);
-  if (sym_prefix && sym_prefix != sym->name[0]
+  if ((sym_prefix && sym_prefix != sym->name[0])
   /*
    * GCC may add special symbols to help gdb figure out the file
    * language.  We want to ignore these, since sometimes they mask
@@ -332,7 +332,6 @@ void
 DEFUN (core_create_function_syms, (core_bfd), bfd * core_bfd)
 {
   bfd_vma min_vma = ~0, max_vma = 0;
-  const char *filename, *func_name;
   int class;
   long i, j, found, skip;
 
@@ -431,24 +430,28 @@ DEFUN (core_create_function_syms, (core_bfd), bfd * core_bfd)
        * labels do not appear in the symbol table info, so this isn't
        * necessary.
        */
-      if (get_src_info (symtab.limit->addr, &filename, &func_name,
-                       &symtab.limit->line_num))
-       {
-         symtab.limit->file = source_file_lookup_path (filename);
-
-         if (strcmp (symtab.limit->name, func_name) != 0)
-           {
-             /*
-              * The symbol's address maps to a different name, so
-              * it can't be a function-entry point.  This happens
-              * for labels, for example.
-              */
-             DBG (AOUTDEBUG,
-               printf ("[core_create_function_syms: rej %s (maps to %s)\n",
-                       symtab.limit->name, func_name));
-             continue;
-           }
-       }
+      {
+       const char *filename, *func_name;
+       
+       if (get_src_info (symtab.limit->addr, &filename, &func_name,
+                         &symtab.limit->line_num))
+         {
+           symtab.limit->file = source_file_lookup_path (filename);
+
+           if (strcmp (symtab.limit->name, func_name) != 0)
+             {
+               /*
+                * The symbol's address maps to a different name, so
+                * it can't be a function-entry point.  This happens
+                * for labels, for example.
+                */
+               DBG (AOUTDEBUG,
+                    printf ("[core_create_function_syms: rej %s (maps to %s)\n",
+                            symtab.limit->name, func_name));
+               continue;
+             }
+         }
+      }
 #endif
 
       symtab.limit->is_func = TRUE;

diff --git a/gprof/symtab.c b/gprof/symtab.c
index 6fd48c3c8f5e8281545e69d62abceb3421a1c330..b38ae6a62dec86cdb3777e08e7f92acd4b40a1b9 100644 (file)
--- a/gprof/symtab.c
+++ b/gprof/symtab.c
@@ -93,10 +93,10 @@ DEFUN (symtab_finalize, (tab), Sym_Table * tab)
           * symbols (such as __gnu_compiled, __c89_used, etc.).
           */
          if ((!src->is_static && dst[-1].is_static)
-             || ((src->is_static == dst[-1].is_static) &&
-                 (src->name[0] != '_' && dst[-1].name[0] == '_')
-                 || (src->name[0]
-                     && src->name[1] != '_' && dst[-1].name[1] == '_')))
+             || ((src->is_static == dst[-1].is_static)
+                 && ((src->name[0] != '_' && dst[-1].name[0] == '_')
+                     || (src->name[0]
+                         && src->name[1] != '_' && dst[-1].name[1] == '_'))))
            {
              DBG (AOUTDEBUG | IDDEBUG,
                   printf ("[symtab_finalize] favor %s@%c%c over %s@%c%c",