This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [PATCH] Re: fold_builtin_memcpy cannot optimize <retval>? (take2)

From: Mark Mitchell <mark at codesourcery dot com>
To: Jakub Jelinek <jakub at redhat dot com>
Cc: law at redhat dot com, Richard Guenther <rguenth at tat dot physik dot uni-tuebingen dot de>, gcc-patches at gcc dot gnu dot org
Date: Fri, 21 Jan 2005 14:03:12 -0800
Subject: Re: [PATCH] Re: fold_builtin_memcpy cannot optimize <retval>? (take2)
Organization: CodeSourcery, LLC
References: <Pine.LNX.4.44.0501211136170.16464-100000@alwazn.tat.physik.uni-tuebingen.de> <20050121105158.GH4777@sunsite.mff.cuni.cz> <1106330271.30575.265.camel@localhost.localdomain> <41F14409.8000200@codesourcery.com> <20050121215138.GH10340@devserv.devel.redhat.com>

Jakub Jelinek wrote:

Is this ok to commit then assuming bootstrap/regtest are ok?
Certainly cures the testcase.

+	case POINTER_TYPE:
+	  /* If no parameters remain or the parameter's code does not
+	     match the specified code, return false.  Otherwise continue
+	     checking any remaining arguments.  POINTER_TYPE is handled
+	     specially, because REFERENCE_TYPE should match it as well.  */
+	  if (arglist == 0
+	      || ! POINTER_TYPE_P (TREE_TYPE (TREE_VALUE (arglist))))
+	    goto end;
+	  break;

Just fold this into the "default" case. OK with that change.

Thanks,

--
Mark Mitchell
CodeSourcery, LLC
mark@codesourcery.com
(916) 791-8304

References:
- [PATCH] Re: fold_builtin_memcpy cannot optimize <retval>? (take 2)
  - From: Jakub Jelinek

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]