diff mbox

Make rs6000 port bootstrap using G++ as 2nd/3rd stage compilers

Message ID 20110720180244.GA12585@hungry-tiger.westford.ibm.com
State New
Headers show

Commit Message

Michael Meissner July 20, 2011, 6:02 p.m. UTC 
I tried building the powerpc64-linux compiler today, and it would not
bootstrap, since evidently stages 2 and 3 are built with G++ instead of C, and
G++ is more strict about const pointers.

This patch allows the compiler to bootstrap.  Is it ok to install?

2011-07-20  Michael Meissner  <meissner@linux.vnet.ibm.com>

	* config/rs6000/rs6000.c (rs6000_xcoff_strip_dollar): Rewrite to
	avoid warnings when GCC is built with a C++ compiler.
diff mbox

Patch

Index: gcc/config/rs6000/rs6000.c
===================================================================
--- gcc/config/rs6000/rs6000.c	(revision 176521)
+++ gcc/config/rs6000/rs6000.c	(working copy)
@@ -21893,8 +21893,9 @@  toc_hash_eq (const void *h1, const void 
 const char *
 rs6000_xcoff_strip_dollar (const char *name)
 {
-  char *strip, *p;
-  int len;
+  char *strip;
+  const char *p;
+  size_t len, i;
 
   p = strchr (name, '$');
 
@@ -21902,13 +21903,11 @@  rs6000_xcoff_strip_dollar (const char *n
     return name;
 
   len = strlen (name);
-  strip = (char *) alloca (len + 1);
-  strcpy (strip, name);
-  p = strchr (strip, '$');
-  while (p)
+  strip = XALLOCAVEC (char, len);
+  for (i = 0; i < len; i++)
     {
-      *p = '_';
-      p = strchr (p + 1, '$');
+      int ch = name[i];
+      strip[i] = (ch == '$') ? '_' : ch;
     }
 
   return ggc_alloc_string (strip, len);