Fix memory report layout at various places.

	* alloc-pool.h (struct pool_usage): Correct space padding.
	* ggc-page.c (ggc_print_statistics): Align columns in a report.
	* mem-stats.h (struct mem_usage): Add argument to print_dash_line.
	* tree.c (dump_tree_statistics): Align columns in a report.

From-SVN: r224071

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 0a18c89d5606def4f9f057d5f47335f32b1c4831..9f054ff145724bc7470451b3531e05f7bb57bb4d 100644 (file)
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,10 @@
+2015-06-03  Martin Liska  <mliska@suse.cz>
+
+       * alloc-pool.h (struct pool_usage): Correct space padding.
+       * ggc-page.c (ggc_print_statistics): Align columns in a report.
+       * mem-stats.h (struct mem_usage): Add argument to print_dash_line.
+       * tree.c (dump_tree_statistics): Align columns in a report.
+
 2015-06-03  Martin Liska  <mliska@suse.cz>
 
        * alloc-pool.c (allocate_pool_descriptor): Remove.

diff --git a/gcc/alloc-pool.h b/gcc/alloc-pool.h
index b1bd386156ba6d812c661276dd470f27723d9109..b10cfa413b844444542c371c3bb923d9ea6eb659 100644 (file)
--- a/gcc/alloc-pool.h
+++ b/gcc/alloc-pool.h
@@ -76,7 +76,7 @@ struct pool_usage: public mem_usage
   inline void dump_footer ()
   {
     print_dash_line ();
-    fprintf (stderr, "%s%75li%10li\n", "Total", (long)m_instances,
+    fprintf (stderr, "%s%82li%10li\n", "Total", (long)m_instances,
             (long)m_allocated);
     print_dash_line ();
   }

diff --git a/gcc/ggc-page.c b/gcc/ggc-page.c
index 158156a37b5368cfc4a037b05dc8a10693628519..7fceeba2a5113338e78c335ed167f0f97b7e4f73 100644 (file)
--- a/gcc/ggc-page.c
+++ b/gcc/ggc-page.c
@@ -2268,7 +2268,7 @@ ggc_print_statistics (void)
      allocation.  */
   fprintf (stderr,
            "Memory still allocated at the end of the compilation process\n");
-  fprintf (stderr, "%-5s %10s  %10s  %10s\n",
+  fprintf (stderr, "%-8s %10s  %10s  %10s\n",
           "Size", "Allocated", "Used", "Overhead");
   for (i = 0; i < NUM_ORDERS; ++i)
     {
@@ -2295,47 +2295,51 @@ ggc_print_statistics (void)
          overhead += (sizeof (page_entry) - sizeof (long)
                       + BITMAP_SIZE (OBJECTS_IN_PAGE (p) + 1));
        }
-      fprintf (stderr, "%-5lu %10lu%c %10lu%c %10lu%c\n",
+      fprintf (stderr, "%-8lu %10lu%c %10lu%c %10lu%c\n",
               (unsigned long) OBJECT_SIZE (i),
               SCALE (allocated), STAT_LABEL (allocated),
               SCALE (in_use), STAT_LABEL (in_use),
               SCALE (overhead), STAT_LABEL (overhead));
       total_overhead += overhead;
     }
-  fprintf (stderr, "%-5s %10lu%c %10lu%c %10lu%c\n", "Total",
+  fprintf (stderr, "%-8s %10lu%c %10lu%c %10lu%c\n", "Total",
           SCALE (G.bytes_mapped), STAT_LABEL (G.bytes_mapped),
           SCALE (G.allocated), STAT_LABEL (G.allocated),
           SCALE (total_overhead), STAT_LABEL (total_overhead));
 
   if (GATHER_STATISTICS)
     {
-      fprintf (stderr, "\nTotal allocations and overheads during the compilation process\n");
+      fprintf (stderr, "\nTotal allocations and overheads during "
+              "the compilation process\n");
 
-      fprintf (stderr, "Total Overhead:                        %10" HOST_LONG_LONG_FORMAT "d\n",
-              G.stats.total_overhead);
-      fprintf (stderr, "Total Allocated:                       %10" HOST_LONG_LONG_FORMAT "d\n",
+      fprintf (stderr, "Total Overhead:                          %10"
+              HOST_LONG_LONG_FORMAT "d\n", G.stats.total_overhead);
+      fprintf (stderr, "Total Allocated:                         %10"
+              HOST_LONG_LONG_FORMAT "d\n",
               G.stats.total_allocated);
 
-      fprintf (stderr, "Total Overhead  under  32B:            %10" HOST_LONG_LONG_FORMAT "d\n",
-              G.stats.total_overhead_under32);
-      fprintf (stderr, "Total Allocated under  32B:            %10" HOST_LONG_LONG_FORMAT "d\n",
-              G.stats.total_allocated_under32);
-      fprintf (stderr, "Total Overhead  under  64B:            %10" HOST_LONG_LONG_FORMAT "d\n",
-              G.stats.total_overhead_under64);
-      fprintf (stderr, "Total Allocated under  64B:            %10" HOST_LONG_LONG_FORMAT "d\n",
-              G.stats.total_allocated_under64);
-      fprintf (stderr, "Total Overhead  under 128B:            %10" HOST_LONG_LONG_FORMAT "d\n",
-              G.stats.total_overhead_under128);
-      fprintf (stderr, "Total Allocated under 128B:            %10" HOST_LONG_LONG_FORMAT "d\n",
-              G.stats.total_allocated_under128);
+      fprintf (stderr, "Total Overhead  under  32B:              %10"
+              HOST_LONG_LONG_FORMAT "d\n", G.stats.total_overhead_under32);
+      fprintf (stderr, "Total Allocated under  32B:              %10"
+              HOST_LONG_LONG_FORMAT "d\n", G.stats.total_allocated_under32);
+      fprintf (stderr, "Total Overhead  under  64B:              %10"
+              HOST_LONG_LONG_FORMAT "d\n", G.stats.total_overhead_under64);
+      fprintf (stderr, "Total Allocated under  64B:              %10"
+              HOST_LONG_LONG_FORMAT "d\n", G.stats.total_allocated_under64);
+      fprintf (stderr, "Total Overhead  under 128B:              %10"
+              HOST_LONG_LONG_FORMAT "d\n", G.stats.total_overhead_under128);
+      fprintf (stderr, "Total Allocated under 128B:              %10"
+              HOST_LONG_LONG_FORMAT "d\n", G.stats.total_allocated_under128);
 
       for (i = 0; i < NUM_ORDERS; i++)
        if (G.stats.total_allocated_per_order[i])
          {
-           fprintf (stderr, "Total Overhead  page size %7lu:     %10" HOST_LONG_LONG_FORMAT "d\n",
+           fprintf (stderr, "Total Overhead  page size %9lu:     %10"
+                    HOST_LONG_LONG_FORMAT "d\n",
                     (unsigned long) OBJECT_SIZE (i),
                     G.stats.total_overhead_per_order[i]);
-           fprintf (stderr, "Total Allocated page size %7lu:     %10" HOST_LONG_LONG_FORMAT "d\n",
+           fprintf (stderr, "Total Allocated page size %9lu:     %10"
+                    HOST_LONG_LONG_FORMAT "d\n",
                     (unsigned long) OBJECT_SIZE (i),
                     G.stats.total_allocated_per_order[i]);
          }

diff --git a/gcc/mem-stats.h b/gcc/mem-stats.h
index 092bfd3610e1ecd546d7c4564a0c61e166819254..641f7c1c787e345ea9714366d7f66296af270934 100644 (file)
--- a/gcc/mem-stats.h
+++ b/gcc/mem-stats.h
@@ -185,9 +185,9 @@ struct mem_usage
   }
 
   /* Print line made of dashes.  */
-  static inline void print_dash_line ()
+  static inline void print_dash_line (size_t count = 140)
   {
-    fprintf (stderr, "%s\n", std::string (140, '-').c_str ());
+    fprintf (stderr, "%s\n", std::string (count, '-').c_str ());
   }
 
   /* Dump header with NAME.  */

diff --git a/gcc/tree.c b/gcc/tree.c
index 805e2cc42c9b2303457e9066c124d985ecbbb962..701a592cca2816596b0344e23dc37c374ca3b1e7 100644 (file)
--- a/gcc/tree.c
+++ b/gcc/tree.c
@@ -9156,6 +9156,8 @@ get_callee_fndecl (const_tree call)
   return NULL_TREE;
 }
 
+#define TREE_MEM_USAGE_SPACES 40
+
 /* Print debugging information about tree nodes generated during the compile,
    and any language-specific information.  */
 
@@ -9166,8 +9168,8 @@ dump_tree_statistics (void)
     {
       int i;
       int total_nodes, total_bytes;
-      fprintf (stderr, "Kind                   Nodes      Bytes\n");
-      fprintf (stderr, "---------------------------------------\n");
+      fprintf (stderr, "\nKind                   Nodes      Bytes\n");
+      mem_usage::print_dash_line (TREE_MEM_USAGE_SPACES);
       total_nodes = total_bytes = 0;
       for (i = 0; i < (int) all_kinds; i++)
        {
@@ -9176,17 +9178,20 @@ dump_tree_statistics (void)
          total_nodes += tree_node_counts[i];
          total_bytes += tree_node_sizes[i];
        }
-      fprintf (stderr, "---------------------------------------\n");
+      mem_usage::print_dash_line (TREE_MEM_USAGE_SPACES);
       fprintf (stderr, "%-20s %7d %10d\n", "Total", total_nodes, total_bytes);
-      fprintf (stderr, "---------------------------------------\n");
+      mem_usage::print_dash_line (TREE_MEM_USAGE_SPACES);
       fprintf (stderr, "Code                   Nodes\n");
-      fprintf (stderr, "----------------------------\n");
+      mem_usage::print_dash_line (TREE_MEM_USAGE_SPACES);
       for (i = 0; i < (int) MAX_TREE_CODES; i++)
-       fprintf (stderr, "%-20s %7d\n", get_tree_code_name ((enum tree_code) i),
+       fprintf (stderr, "%-32s %7d\n", get_tree_code_name ((enum tree_code) i),
                  tree_code_counts[i]);
-      fprintf (stderr, "----------------------------\n");
+      mem_usage::print_dash_line (TREE_MEM_USAGE_SPACES);
+      fprintf (stderr, "\n");
       ssanames_print_statistics ();
+      fprintf (stderr, "\n");
       phinodes_print_statistics ();
+      fprintf (stderr, "\n");
     }
   else
     fprintf (stderr, "(No per-node statistics)\n");