C++ new and aliasing

C++ new and aliasing

[John Carr]

This patch fixes a bug in the code which tests whether a call is to
a malloc-like function.  (The alias code knows that the return value
of malloc does not alias any other memory.)


Sat Apr 25 16:38:46 1998  John Carr  <jfc@mit.edu>

	* calls.c (expand_call): Fix code to recognize C++ operator new.

*** calls.c	1998/04/09 00:44:41	1.19
--- calls.c	1998/04/25 20:40:57
*************** expand_call (exp, target, ignore)
*** 901,908 ****
        else if (! strcmp (tname, "malloc")
  	       || ! strcmp (tname, "calloc")
  	       || ! strcmp (tname, "realloc")
! 	       || ! strcmp (tname, "__builtin_new")
! 	       || ! strcmp (tname, "__builtin_vec_new"))
  	is_malloc = 1;
      }
  
--- 901,912 ----
        else if (! strcmp (tname, "malloc")
  	       || ! strcmp (tname, "calloc")
  	       || ! strcmp (tname, "realloc")
! 	       /* Note use of NAME rather than TNAME here.  These functions
! 		  are only reserved when preceded with __.  */
! 	       || ! strcmp (name, "__vn")	/* mangled __builtin_vec_new */
! 	       || ! strcmp (name, "__nw")	/* mangled __builtin_new */
! 	       || ! strcmp (name, "__builtin_new")
! 	       || ! strcmp (name, "__builtin_vec_new"))
  	is_malloc = 1;
      }
  

Re: C++ new and aliasing

[Jim Wilson]

	* calls.c (expand_call): Fix code to recognize C++ operator new.

This is OK to install.

Jim

Re: C++ new and aliasing

[Andreas Schwab]

Here is another bug fix for expand_call: __builtin_vec_new has 17
characters.


1998-04-27  Andreas Schwab  <schwab@issan.informatik.uni-dortmund.de>

	* calls.c (expand_call): Bump the length limit on the specially
	recognized function names to 17.

--- egcs-2.91.25/gcc/calls.c.~1~	Thu Apr 16 00:53:17 1998
+++ egcs-2.91.25/gcc/calls.c	Mon Apr 27 21:30:17 1998
@@ -854,7 +854,7 @@
   is_longjmp = 0;
   is_malloc = 0;
 
-  if (name != 0 && IDENTIFIER_LENGTH (DECL_NAME (fndecl)) <= 15
+  if (name != 0 && IDENTIFIER_LENGTH (DECL_NAME (fndecl)) <= 17
       /* Exclude functions not at the file scope, or not `extern',
 	 since they are not the magic functions we would otherwise
 	 think they are.  */

-- 
Andreas Schwab                                      "And now for something
schwab@issan.informatik.uni-dortmund.de              completely different"
schwab@gnu.org

Re: C++ new and aliasing

[Jim Wilson]

This is OK to install.

Jim