optimization/4815

optimization/4815

[rodrigc]

The following reply was made to PR optimization/4815; it has been noted by GNATS.

From: rodrigc@gcc.gnu.org
To: gcc-gnats@gcc.gnu.org
Cc:  
Subject: optimization/4815
Date: 18 Nov 2001 08:16:38 -0000

 CVSROOT:	/cvs/gcc
 Module name:	gcc
 Branch: 	gcc-3_0-branch
 Changes by:	rodrigc@gcc.gnu.org	2001-11-18 00:16:38
 
 Modified files:
 	gcc            : ChangeLog alias.c 
 
 Log message:
 	2001-11-18  Craig Rodrigues  <rodrigc@gcc.gnu.org>
 	
 	PR optimization/4815
 	* alias.c: Merge from mainline:
 	2001-08-05  Bernd Schmidt  <bernds@redhat.com>
 	* alias.c (rtx_equal_for_memref_p): VALUEs are only identical
 	if their CSELIB_VAL_PTRs are.
 	2001-07-25  Andrew Haley  <aph@cambridge.redhat.com>
 	* alias.c (rtx_equal_for_memref_p): Allow strings as types in
 	operands.
 
 Patches:
 http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/ChangeLog.diff?cvsroot=gcc&only_with_tag=gcc-3_0-branch&r1=1.9240.2.912&r2=1.9240.2.913
 http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/alias.c.diff?cvsroot=gcc&only_with_tag=gcc-3_0-branch&r1=1.115.4.7&r2=1.115.4.8
 

Re: optimization/4815

[Craig Rodrigues]

The following reply was made to PR optimization/4815; it has been noted by GNATS.

From: Craig Rodrigues <rodrigc@mediaone.net>
To: gcc-gnats@gcc.gnu.org, gcc-prs@gcc.gnu.org,
   ecoffey@students.latrobe.edu.au, gcc-bugs@gcc.gnu.org, nobody@gcc.gnu.org
Cc:  
Subject: Re: optimization/4815
Date: Sat, 17 Nov 2001 01:21:41 -0500

 http://gcc.gnu.org/cgi-bin/gnatsweb.pl?cmd=view%20audit-trail&pr=4815&database=gcc
 
 I have a better patch than the one mentioned in the e-mail archive.
 
 Can you try the following patch?  It merges in some changes from the gcc
 3.1
 tree.
 
 Index: alias.c
 ===================================================================
 RCS file: /cvs/gcc/gcc/gcc/alias.c,v
 retrieving revision 1.115.4.7
 diff -u -r1.115.4.7 alias.c
 --- alias.c     2001/09/11 21:39:24     1.115.4.7
 +++ alias.c     2001/11/17 06:19:01
 @@ -1041,6 +1041,9 @@
    /* Some RTL can be compared without a recursive examination.  */
    switch (code)
      {
 +    case VALUE:
 +     return CSELIB_VAL_PTR (x) == CSELIB_VAL_PTR (y);
 +
      case REG:
        return REGNO (x) == REGNO (y);
 
 @@ -1109,6 +1112,12 @@
           if (rtx_equal_for_memref_p (XEXP (x, i), XEXP (y, i)) == 0)
             return 0;
           break;
 +
 +         /* This can happen for asm operands.  */
 +       case 's':
 +         if (strcmp (XSTR (x, i), XSTR (y, i)))
 +           return 0;
 +       break;
 
         /* This can happen for an asm which clobbers memory.  */
         case '0':
 
 --
 Craig Rodrigues
 http://www.gis.net/~craigr
 rodrigc@mediaone.net