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] improve C++ code by changing fold-const.c

From: Andrew Pinski <pinskia at physics dot uc dot edu>
To: Jeff Law <law at redhat dot com>, Dale Johannesen <dalej at apple dot com>, Diego Novillo <dnovillo at redhat dot com>
Cc: Roger Sayle <roger at eyesopen dot com>, gcc-patches at gcc dot gnu dot org, Richard Henderson <rth at redhat dot com>, Mark Mitchell <mark at codesourcery dot com>, Andrew Pinski <pinskia at physics dot uc dot edu>
Date: Wed, 2 Jun 2004 00:30:01 -0400
Subject: Re: [PATCH] improve C++ code by changing fold-const.c
References: <200405210300.i4L30AW5011486@speedy.slc.redhat.com> <40AEB4C1.6090804@codesourcery.com> <4E341518-AB9A-11D8-A4A1-000393A6D2F2@physics.uc.edu> <40AECA32.60900@codesourcery.com> <B47F4926-AF70-11D8-ACD4-000393A6D2F2@physics.uc.edu> <1086124024.2870.5.camel@speedy>

On Jun 2, 2004, at 00:12, Jeffrey A Law wrote:

Basically if the inner & outer types are pointer types (which include
references), then we want to compare what each type points to.  This
(of course) iterates until one or both types are no longer a pointer
type.

Actually it looks like Dale messed up when he changed tree_ssa_useless_type_conversion_1 to use the language hook and he changed the checking of the how it considers the pointer types compatible, which broke this tree_ssa_useless_type_conversion with this respect. See <http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/tree-ssa.c.diff? r1=1.1.4.212&r2=1.1.4.213&f=h>


Thanks,
Andrew Pinski

Follow-Ups:
- Re: [PATCH] improve C++ code by changing fold-const.c
  - From: Andrew Pinski

References:
- Re: [PATCH] improve C++ code by changing fold-const.c
  - From: Jeffrey A Law

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