2011-04-29 Andrew Gontarek <andrewg@cray.com>

	* valprint.c (val_print_array_elements): Fixed poor performance
	of printing very large arrays with repeat_count_threshold set
	to unlimited.  New comment.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 9910084b30e8876babec3b483c8d814cbf62beb9..4d33271890c828eb63266a07f08b00163ecde898 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,9 @@
+2011-04-29  Andrew Gontarek  <andrewg@cray.com>
+
+       * valprint.c (val_print_array_elements): Fixed poor performance
+       of printing very large arrays with repeat_count_threshold set
+       to unlimited.  New comment.
+
 2011-04-29  Pedro Alves  <pedro@codesourcery.com>
 
        * linux-nat.c (linux_child_remove_fork_catchpoint)

diff --git a/gdb/valprint.c b/gdb/valprint.c
index 286ef9e74cd4b7d29310c27b0982bd37ac308c74..9bf19f4b469bc31912b9ece503f2badd5e793491 100644 (file)
--- a/gdb/valprint.c
+++ b/gdb/valprint.c
@@ -1247,15 +1247,21 @@ val_print_array_elements (struct type *type,
 
       rep1 = i + 1;
       reps = 1;
-      while (rep1 < len
-            && value_available_contents_eq (val,
-                                            embedded_offset + i * eltlen,
-                                            val,
-                                            embedded_offset + rep1 * eltlen,
-                                            eltlen))
+      /* Only check for reps if repeat_count_threshold is not set to
+        UINT_MAX (unlimited).  */
+      if (options->repeat_count_threshold < UINT_MAX)
        {
-         ++reps;
-         ++rep1;
+         while (rep1 < len
+                && value_available_contents_eq (val,
+                                                embedded_offset + i * eltlen,
+                                                val,
+                                                (embedded_offset
+                                                 + rep1 * eltlen),
+                                                eltlen))
+           {
+             ++reps;
+             ++rep1;
+           }
        }
 
       if (reps > options->repeat_count_threshold)